-
Notifications
You must be signed in to change notification settings - Fork 4
refactor: final v11 release #56
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
Conversation
e7062aa to
00d3f6f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR performs a comprehensive rebranding/terminology update, transitioning from "parachain" to "chain" throughout the documentation, along with cleaning up outdated content related to Pop Network and ink! v5/v6 migration guides.
Key changes:
- Terminology shift from "parachain" to "chain" across all documentation files
- Removal of deprecated content (Pop Network tutorials, ink! v6 migration guide, PSP22/PSP34 token tutorials)
- Restructured installation and hackathon guides with updated version information
- Updated file references and paths to match new naming conventions
Reviewed Changes
Copilot reviewed 37 out of 38 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| welcome/install-pop-cli.md | Restructured installation guide with better hierarchy, updated ink! version guidance, and added help section |
| welcome/hackathon-guide.md | Reordered contract/chain sections, updated terminology from "parachain" to "chain", updated file path references |
| welcome/get-tokens-on-pop-network.md | File deleted - Pop Network being sunset |
| pop-network/tutorials/your-first-contract.md | Updated command help text from "parachain" to "chain" |
| pop-cli-for-smart-contracts/welcome/migrating-to-inkv6.md | File deleted - ink! v6 is now default, migration guide no longer needed |
| pop-cli-for-smart-contracts/welcome/install-pop-cli.md | Updated installation instructions, added ink! version guidance and help section |
| pop-cli-for-smart-contracts/tutorials/*.md | Deleted obsolete tutorials (your-first-ink-smart-contract, PSP22/PSP34 minting) |
| pop-cli-for-smart-contracts/guides/*.md | Removed ink! v6 migration references, cleaned up outdated notes and images |
| pop-cli-for-smart-contracts/SUMMARY.md | Removed deleted tutorial references and ink! v6 migration guide link |
| pop-cli-for-appchains/*.md | Comprehensive "parachain" to "chain" terminology updates across all guides |
| pop-cli-for-appchains/guides/create-a-new-parachain/*.md | Directory structure removed, replaced with single create-a-new-chain.md |
| pop-cli-for-appchains/SUMMARY.md | Updated navigation structure to reflect new file organization |
| .gitignore | Added .DS_Store to ignore list |
| README.md | Added image reference for hackathon promotion |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
pop-cli-for-appchains/guides/launch-a-chain/launch-a-known-chain.md
Outdated
Show resolved
Hide resolved
pop-cli-for-appchains/guides/launch-a-chain/launch-a-known-chain.md
Outdated
Show resolved
Hide resolved
pop-cli-for-appchains/guides/launch-a-chain/launch-a-known-chain.md
Outdated
Show resolved
Hide resolved
…in.md Co-authored-by: Copilot <[email protected]>
…in.md Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
…in.md Co-authored-by: Copilot <[email protected]>
al3mart
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems alright 👍
AlexD10S
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Cleaner also think remove the tutorials is a good idea
* feat: update pop-install instructions from macos (#53) * feat: update docs for pop with latest changes (#55) * feat: update docs for pop call chain * feat: update docs about pop build spec * refactor: remove unused images * feat: update pop build spec docs * feat: add instruction on how to install from brew * refactor: address PR feedback * fix: review feedback * refactor: final v11 release (#56) * refactor: final v11 changes * refactor: remove tutorial * refactor: create new chain * refactor: chain terminology * refactor: remove gifs * Update pop-cli-for-appchains/guides/launch-a-chain/launch-a-known-chain.md Co-authored-by: Copilot <[email protected]> * Update pop-cli-for-appchains/guides/launch-a-chain/launch-a-known-chain.md Co-authored-by: Copilot <[email protected]> * Update README.md Co-authored-by: Copilot <[email protected]> * Update pop-cli-for-appchains/guides/launch-a-chain/launch-a-known-chain.md Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: José Molina Colmenero <[email protected]> Co-authored-by: Daan van der Plas <[email protected]> Co-authored-by: Copilot <[email protected]>
chain