Skip to content

Conversation

@JonesRoy3
Copy link

This PR adds an internal documentation page under docs/ that explains how to contribute to ethereum.org using only the GitHub web UI, without setting up a local environment. The guide summarizes when this workflow is appropriate, its limitations, and step-by-step instructions for editing or creating Markdown files directly on GitHub.

This PR adds an internal documentation page under docs/ that explains how to contribute to ethereum.org using only the GitHub web UI, without setting up a local environment. The guide summarizes when this workflow is appropriate, its limitations, and step-by-step instructions for editing or creating Markdown files directly on GitHub.
@netlify
Copy link

netlify bot commented Dec 2, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit dc12de3
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/692ed0144d0f5d0008347bcf
😎 Deploy Preview https://deploy-preview-16764--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 42 (🔴 down 17 from production)
Accessibility: 94 (no change from production)
Best Practices: 92 (🔴 down 7 from production)
SEO: 91 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the documentation 📖 Change or add documentation label Dec 2, 2025
Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Looks good overall, thanks @JonesRoy3... just a few requests left below =)

Worth also considering how to surface this for users at this level so they can find this content

4. Scroll down to the **Commit changes** section:
- Write a short, descriptive commit message (e.g. `Fix typo in nodes and clients docs`).
- Optionally add a longer description explaining what you changed and why.
5. Choose **“Commit directly to this branch”** (or create a new branch if you are working from your fork).
Copy link
Member

Choose a reason for hiding this comment

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

Would update to recommend a new branch with a simple name

6. Commit to your feature branch.
7. Open a pull request to the upstream `dev` branch.

For content that appears on ethereum.org (not just internal docs), we usually keep Markdown files under `public/content/` and follow the structure explained in `docs/best-practices.md`. This document is meant to complement that guide, not replace it.
Copy link
Member

Choose a reason for hiding this comment

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

Markdown content is always in this folder. Would update to account for this, since JSON strings and TypeScript files are not recommended for this approach

1. Navigate to the folder where the file should live (e.g. `docs/`).
2. Click **Add file → Create new file**.
3. Enter a file name, such as `docs/my-new-guide.md` or `my-new-guide.md` if you are already inside the `docs/` folder.
4. Paste your Markdown content into the editor.
Copy link
Member

Choose a reason for hiding this comment

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

Probably worth noting that front matter is needed for all markdown content files... it can get nuanced, so probably best to suggest looking at other files as examples

- Prefer **small, focused PRs** that do one thing well.
- Double-check that you didn't accidentally edit unrelated lines when using the web editor.
- Use the **Preview** tab in the GitHub editor to verify Markdown formatting.
- If you are unsure whether a change is appropriate, open an issue first and ask for feedback.
Copy link
Member

Choose a reason for hiding this comment

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

Especially for new pages, an issue should be opened ahead of time

- You cannot run automated tests before pushing changes.
- Large refactors are harder to review and reason about.

For frequent contributors or more complex work, we recommend switching to a full local development environment using `pnpm`, as documented in the main repository README.
Copy link
Member

Choose a reason for hiding this comment

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

Would scrape "using pnpm" comment, and turn README into a link

@corwintines
Copy link
Member

Overall not against this, but its a burried file that is unlikely to get seen so this seems a bit unnecessary IMO. Most of this can be looked up about github since its specific to github.

Won't veto it completely, but changes @wackerow had should be addressed first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation 📖 Change or add documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants