Skip to content

chore: update nv27 codename#13334

Merged
rjan90 merged 2 commits into
masterfrom
phi/update-nv27-codename
Sep 11, 2025
Merged

chore: update nv27 codename#13334
rjan90 merged 2 commits into
masterfrom
phi/update-nv27-codename

Conversation

@rjan90
Copy link
Copy Markdown
Contributor

@rjan90 rjan90 commented Sep 11, 2025

Proposed Changes

Updates the nv27 codename to Goldenweek, to be in line with filecoin-project/community#74 (comment)

Additional Info

Checklist

Before you mark the PR ready for review, please make sure that:

chore: update nv27 codename
@rjan90 rjan90 self-assigned this Sep 11, 2025
Copilot AI review requested due to automatic review settings September 11, 2025 16:28
@rjan90 rjan90 added release/backport skip/changelog This change does not require CHANGELOG.md update labels Sep 11, 2025
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz Sep 11, 2025
@rjan90 rjan90 moved this from 📌 Triage to 🔎 Awaiting Review in FilOz Sep 11, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Updates the Filecoin network version 27 (nv27) codename from "Xx" to "Goldenweek" to align with community discussions and official naming conventions.

  • Renames all references to UpgradeXxHeight to UpgradeGoldenweekHeight across configuration files
  • Updates corresponding environment variable names and documentation
  • Modifies API response structures to reflect the new upgrade name

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
node/impl/full/state.go Updates API response to use new upgrade field name
documentation/en/api-methods-v1-stable.md Updates API documentation example response
documentation/en/api-methods-v0-deprecated.md Updates deprecated API documentation example response
chain/consensus/filcns/upgrades.go Updates upgrade configuration to reference new height constant
build/openrpc/gateway.json Updates OpenRPC schema with new upgrade field name
build/buildconstants/params_*.go Renames upgrade height constants and environment variables across all network configurations
api/types.go Updates ForkUpgradeParams struct field name

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Copy Markdown
Member

@BigLep BigLep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Just need to change to UpgradeGoldenWeekHeight per verbal to be consistent.

@github-project-automation github-project-automation Bot moved this from 🔎 Awaiting Review to ✔️ Approved by reviewer in FilOz Sep 11, 2025
chore: update nv27 codename
@rjan90 rjan90 merged commit 9e5d3f4 into master Sep 11, 2025
98 checks passed
@rjan90 rjan90 deleted the phi/update-nv27-codename branch September 11, 2025 16:55
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Sep 11, 2025
BigLep pushed a commit that referenced this pull request Sep 11, 2025
* chore: update nv27 codename

chore: update nv27 codename

* chore: update nv27 codename

chore: update nv27 codename
BigLep added a commit that referenced this pull request Sep 11, 2025
* chore: update nv27 codename (#13334)

* chore: update nv27 codename

chore: update nv27 codename

* chore: update nv27 codename

chore: update nv27 codename

* chore: update Mainnet Golden Week height (#13335)

```
2025-09-24T23:00:00Z
5348280
```

* chore: update deps for NV27 final release (#13336)

* chore: update FFI

* Add go-state-types and builtin-actors deps

* Update actor bundles

---------

Co-authored-by: TippyFlits <james@filoz.org>

* chore: update filecoin-ffi submodule to match v1.34.0 dependency

---------

Co-authored-by: Phi-rjan <orjan.roren@gmail.com>
Co-authored-by: TippyFlits <james@filoz.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

4 participants