Skip to content

Conversation

@princetonbishop
Copy link
Collaborator

@princetonbishop princetonbishop commented Jan 27, 2026

Description

What does this PR solve?

Fixes # (issue)

Checklist

  • Code follows the style guide
  • I have performed a self-review of my own code
  • New and existing tests pass locally
  • This PR is targeting the correct branch

Summary by CodeRabbit

  • Chores
    • Updated contract addresses for mainnet-templegold configuration
    • Reorganized deployment configuration files for sepolia network
    • Updated factory implementation reference in deployment scripts

✏️ Tip: You can customize this high-level summary in your review settings.

@princetonbishop princetonbishop requested a review from a team as a code owner January 27, 2026 21:25
@coderabbitai
Copy link

coderabbitai bot commented Jan 27, 2026

📝 Walkthrough

Walkthrough

This pull request updates deployment configuration for the templegold protocol on both mainnet and Sepolia networks. The mainnet contract-addresses.ts file is updated with new addresses for SPICE_AUCTION_IMPLEMENTATION, SPICE_TOKEN, and SPICE_TGLD. Multiple Sepolia deployment argument files are removed, while two new ones are added with updated constructor parameters. Additionally, the Sepolia spice-auction-implementation deployment script switches from using SpiceAuction__factory to SpiceMock__factory for instantiation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested labels

preview-deploy

🚥 Pre-merge checks | ❌ 3
❌ Failed checks (2 warnings, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is completely empty of meaningful content, containing only an unfilled template with placeholder text 'Fixes # (issue)' and unchecked checklist items, with no explanation of what the PR solves or what changes were made. Fill in the description with details about what this PR solves, which deployment addresses were updated, which files were added/removed, and why these changes are necessary.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'deploy scripts' is vague and generic, using non-descriptive terms that don't convey the specific nature of the substantial changes made to deployment configuration and contract implementations. Use a more specific title that describes the key changes, such as 'Update mainnet-templegold deployment contract addresses and Sepolia deployment arguments' or 'Refactor deployment scripts for templegold migration'.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

@princetonbishop princetonbishop merged commit f5dbb77 into stage Jan 28, 2026
6 checks passed
@princetonbishop princetonbishop deleted the sepolia-deploy-update branch January 28, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants