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

Add CONTRIBUTING.md and Contributing Section to README.md #4539

Merged
merged 5 commits into from
Mar 15, 2024

Conversation

richarddushime
Copy link
Contributor

In this pull request, I have made improvements to the main repository by adding a CONTRIBUTING.md file and updating the main README.md to include a small section for contributing.

@richarddushime richarddushime requested a review from a team as a code owner March 12, 2024 08:16
CONTRIBUTING.md Outdated

- **[Development Environment Setup Guide](https://github.com/fedimint/fedimint/blob/master/docs/contributing.md)**: Follow this guide to set up your development environment.

If you have any questions, don't hesitate to reach out to us on our [Developer Discord](https://chat.fedimint.org). Our community is friendly and supportive, and we're here to help you succeed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we have a channel dedicated to new contributors or is general good enough for it?

Copy link
Member

Choose a reason for hiding this comment

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

I made one #new-contributors

Copy link
Member

Choose a reason for hiding this comment

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

just update that link so they land on that channel specifically then it's good

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've refreshed the link with a new invitation that I just created from the #new-contributors channel.

Copy link
Contributor

@dpc dpc left a comment

Choose a reason for hiding this comment

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

LGTM, but @Kodylow should ✔️ it before landing.

CONTRIBUTING.md Outdated

Welcome to the Fedimint project! We're thrilled that you're interested in contributing to our open-source community. Below are some resources to help you get started:

- **[Contributing Guidelines](https://github.com/fedimint/fedimint/blob/master/docs/contributing.md)**: Read our guidelines to understand how you can contribute to Fedimint.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's not have this file link to itself

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The document mentioned is in a different folder from the CONTRIBUTING.md file in the main repository.

Copy link
Member

Choose a reason for hiding this comment

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

just move the contributing.md out of the docs folder into this root, link to it from the readme, and update the instructions to new contributors to include what I put in the discord under the new-contributors channel

Copy link
Contributor

Choose a reason for hiding this comment

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

Afaik having it in the root is better anyway since GitHub can detect it and might have some current or future integrations with it.

CONTRIBUTING.md Outdated

- **[Development Environment Setup Guide](https://github.com/fedimint/fedimint/blob/master/docs/dev-env.md)**: Follow this guide to set up your development environment.

If you have any questions, don't hesitate to reach out to us on our [Developer Discord](https://discord.gg/BGFMXSkNJW). Our community is friendly and supportive, and we're here to help you succeed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this a special link that leads directly into a channel? Otherwise I'd propose using https://chat.fedimint.org, which is easier to change should we ever need a new invite link.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that link leads directly to #new-contributors channel
while the https://chat.fedimint.org leads to general

@dpc
Copy link
Contributor

dpc commented Mar 15, 2024

Everybody got to point out their nits and wishes, don't be afraid to ping me and other directly when addresses and it's squashed. Let's land it soon, as it's a solid improvement, will not break anything one way or the other, so no need to keep it in progress for too long.

@dpc dpc enabled auto-merge March 15, 2024 15:39
@dpc dpc added this pull request to the merge queue Mar 15, 2024
Merged via the queue into fedimint:master with commit e89e376 Mar 15, 2024
20 checks passed
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.

None yet

5 participants