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

fix(ICRC_Ledger): FI-1654: Update link to script that downloads latest ICRC ledger #3977

Merged

Conversation

mbjorkqvist
Copy link
Member

The script that downloads the latest ICRC ledger wasm and did files was recently updated to download them using the official ledger releases, rather than the latest commit on the master branch of the ic repository.

The link to the script in the documentation points to a specific commit in the ic repository. This PR proposes to update the link to the commit where the script was updated, so that users end up with an officially released ledger.

@github-actions github-actions bot added the documentation Changes to Developer Docs label Jan 24, 2025
@mbjorkqvist
Copy link
Member Author

I made a corresponding PR in the examples repo, since the documentation under docs/current/references/samples. e.g., this one, is pulled from there as a submodule. I also updated submodules/samples in this PR, since an earlier PR in the examples repo that also touched the same link as is being updated in this PR hasn't been deployed yet. If there's another process for updating the submodules, please let me know, and I'll revert that commit!

@mbjorkqvist mbjorkqvist marked this pull request as ready for review January 24, 2025 13:20
@mbjorkqvist mbjorkqvist requested a review from a team as a code owner January 24, 2025 13:20
@mbjorkqvist mbjorkqvist merged commit 2ec2f2f into master Jan 24, 2025
5 checks passed
@mbjorkqvist mbjorkqvist deleted the mathias-FI-1654-update-links-to-icrc-download-script branch January 24, 2025 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Changes to Developer Docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants