Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: remove hardcoded versions in chain-upgrade docs #26

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented Aug 6, 2024

Back when the guide was managed in the Penumbra protocol repo [0], the version strings in the docs were automatically bumped in line with the source code changes. Since breaking the guide out into a separate repo, we no longer have the convenience of those automatic updates, so the guide versions will necessarily lag behind. Let's remove the hardcoded version strings from crucial reference pages like the chain-upgrade docs, leaning on node operators to choose the correct version for a given upgrade.

[0] https://github.com/penumbra-zone/penumbra

Back when the guide was managed in the Penumbra protocol repo [0],
the version strings in the docs were automatically bumped in line with
the source code changes. Since breaking the guide out into a separate
repo, we no longer have the convenience of those automatic updates, so
the guide versions will necessarily lag behind. Let's remove the
hardcoded version strings from crucial reference pages like the
chain-upgrade docs, leaning on node operators to choose the correct
version for a given upgrade.

[0] https://github.com/penumbra-zone/penumbra
Copy link

github-actions bot commented Aug 6, 2024

Visit the preview URL for this PR (updated for commit 8cf94a9):

https://penumbra-guide--pr26-remove-hardcoded-ver-f0vpfanh.web.app

(expires Tue, 13 Aug 2024 17:08:51 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c64218fe717b75c9724a23946df883a1adf10faa

@conorsch conorsch merged commit 891e4c8 into main Aug 6, 2024
2 checks passed
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.

2 participants