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

Updates CONTRIBUTING.md #93

Merged
merged 9 commits into from
Feb 1, 2024
Merged

Conversation

arthurgousset
Copy link
Contributor

@arthurgousset arthurgousset commented Jan 24, 2024

Description

Adds a CONTRIBUTING.md file to make it easy for first-time visitors to contribute. I took inspiration from the CONTRIBUTING.md files from foundry and viem.

Tasks

Other changes

Moves CONTRIBUTING.md file from the /.github directory to the top-level to make it easier for new contributors to find it. It will still be recognised by Github for various use cases.

Deleted README-DEV.md and SETUP.md, which are files that were brought over from the monorepo, and are now either out of date or not completely relevant. To avoid confusion, I removed these, but we can always complement the CONTRIBUTING.md file if needed.

Reviewed RELEASE.md, which is a file for internal use (i.e. 3rd-party contributors won't need this) that describes the release process. Fixed a couple typos to improve readability and reviewed the contents tie with our CONTRIBUTING.md.

Tested

N/A, only docs changes.

Related issues

Backwards compatibility

N/A, only docs changes

Documentation

Yes, docs are updated with this PR.

Copy link

changeset-bot bot commented Jan 24, 2024

⚠️ No Changeset found

Latest commit: c174d9a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Doesn't seem relevant for our current workflows anymore.
So it's easier for first-time contributors to find it when visiting the GitHub repo on the browser.
This file was brought from the monorepo and does not seem entirely relevant for @celo-org/developer-tooling. Instead the `CONTRIBUTING.md` file should be expanded if necessary. At this moment, I don't think this is necessary.
@arthurgousset arthurgousset marked this pull request as ready for review January 31, 2024 18:38
@arthurgousset arthurgousset requested a review from a team as a code owner January 31, 2024 18:38
@arthurgousset arthurgousset enabled auto-merge (squash) January 31, 2024 19:50
@arthurgousset
Copy link
Contributor Author

I'll add a couple smaller edits to README

@arthurgousset arthurgousset enabled auto-merge (squash) February 1, 2024 12:46
Copy link
Contributor

@nicolasbrugneaux nicolasbrugneaux left a comment

Choose a reason for hiding this comment

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

Nice work!

@arthurgousset arthurgousset merged commit 2c20aa3 into master Feb 1, 2024
17 checks passed
@arthurgousset arthurgousset deleted the arthurgousset/contribution-guide branch February 1, 2024 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants