Skip to content

Add node and runtime page#369

Merged
eshaben merged 14 commits intomasterfrom
0xlucca/runtime-and-node
Feb 25, 2025
Merged

Add node and runtime page#369
eshaben merged 14 commits intomasterfrom
0xlucca/runtime-and-node

Conversation

@0xLucca
Copy link
Contributor

@0xLucca 0xLucca commented Feb 6, 2025

This PR adds Node and Runtime page in Polkadot Protocol -> Basics section

@0xLucca 0xLucca requested a review from a team as a code owner February 6, 2025 19:10
@0xLucca 0xLucca added B0 - Needs Review Pull request is ready for review C0 - Low Low priority task A0 - New Content Pull request contains new content pages labels Feb 6, 2025
@0xLucca 0xLucca requested a review from nhussein11 February 7, 2025 11:50
Copy link
Collaborator

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

Needs a few small things. Nice breakdown on this one!

Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
- 'Cryptography': cryptography.md
- 'Data Encoding': data-encoding.md
- 'Chain Data': chain-data.md
- "Node and Runtime": node-and-runtime.md
Copy link
Collaborator

Choose a reason for hiding this comment

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

Will you move this page above interoperability.md?

Copy link
Collaborator

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

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

LGTM! Should require to merge the stuff that exists in #390 before merging this one

Copy link
Collaborator

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

grammarly things

0xLucca and others added 2 commits February 19, 2025 12:11
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
@0xLucca 0xLucca requested a review from dawnkelly09 February 19, 2025 15:22
Copy link
Collaborator

@dawnkelly09 dawnkelly09 left a comment

Choose a reason for hiding this comment

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

🚀 perfecto!

@dawnkelly09 dawnkelly09 added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Feb 25, 2025
@eshaben eshaben merged commit af2c011 into master Feb 25, 2025
2 checks passed
@eshaben eshaben deleted the 0xlucca/runtime-and-node branch February 25, 2025 17:04
Telucero pushed a commit to Telucero/polkadot-docs that referenced this pull request Sep 18, 2025
* Add node and runtime page

* Apply suggestions from code review

Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>

* Apply suggestions

* Update llms.txt

* Apply suggestions from code review

Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>

* Move above Interoperability

* Update llms.txt

* Update llms.txt

* update llms

* llms

* llms

---------

Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Dawn Kelly <83190195+dawnkelly09@users.noreply.github.com>
Co-authored-by: Erin Shaben <eshaben@icloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A0 - New Content Pull request contains new content pages B1 - Ready to Merge Pull request is ready to be merged C0 - Low Low priority task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants