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

SPO Node Installation: update as per main Node Installation doc #1409

Closed
wants to merge 1 commit into from

Conversation

linconvidal
Copy link
Member

Checklist

Updating documentation or Bugfix

  • Add a new step for installing BLST, as described in this guide.
  • Bump the Cabal version to match other sources.

- Add a new step for installing BLST, as described in [this guide](https://developers.cardano.org/docs/get-started/cardano-node/installing-cardano-node/#installing-blst).  
- Bump the Cabal version to match other sources.
@rphair rphair changed the title Update node-installation-process.md SPO Node Installation: update as per main Node Installation doc Feb 25, 2025
rphair
rphair previously approved these changes Feb 25, 2025
Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

blst script seems taken from https://developers.cardano.org/docs/get-started/cardano-node/installing-cardano-node#installing-blst - this doc also confirms latest suggested cabal version.

@rphair rphair added the documentation Improvements or additions to documentation label Feb 25, 2025
@rdlrt
Copy link
Collaborator

rdlrt commented Feb 25, 2025

We were supposed to drop the entire section and place a link to the existing page, to avoid redundant copy and difference in content

@rphair
Copy link
Collaborator

rphair commented Feb 26, 2025

yes but I forget who was supposed to submit the PR to make that change 😅 (I'll tag a copy of this in my inbox & likely do it when clear of some other concurrent things)

@rphair rphair dismissed their stale review February 26, 2025 09:54

proper solution: link to main install document

@rphair
Copy link
Collaborator

rphair commented Feb 26, 2025

@rdlrt - superseded by #1411.
@linconvidal - thanks for drawing attention to this issue. 🙏

@rphair rphair closed this Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants