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

Updated contributing.md links. #3727

Merged
merged 5 commits into from
Jul 2, 2022
Merged

Updated contributing.md links. #3727

merged 5 commits into from
Jul 2, 2022

Conversation

cekickafa
Copy link
Contributor


name: contributing.md update
about: updated gitter.im/grin_community channel links to keybase

  • Changed grin support link to keybase grincoin#support channel
  • Changed Grin lobby link to keybase grincoin#general channel
  • Changed developer channel to keybase grincoin#dev
  • Changed @grincoincil twitter account to reddit/grincoin channel for news and developments.
  • Added doc.grin.mw link for general grin information.

If your PR is a work in progress, please feel free to create it and include a [WIP] tag in the PR name. We encourage everyone to PR early and often so that other developers know what you're working on.

Before submitting your PR for final review, please ensure that it:

  • Includes a proper description of what problems the PR addresses, as well as a detailed explanation as to what it changes
  • Explains whether/how the change is consensus breaking or breaks existing client functionality
  • Contains unit tests exercising new/changed functionality
  • Fully considers the potential impact of the change on other parts of the system
  • Describes how you've tested the change (e.g. against Testnet, etc)
  • Updates any documentation that's affected by the PR

cekickafa added 2 commits July 1, 2022 21:15
- Changed grin support link to keybase grincoin#support channel
- Changed Grin lobby link to keybase grincoin#general channel
- Changed developer channel to keybase grincoin#dev
- Changed @grincoincil twitter account to reddit/grincoin channel for news and developments.
- Added doc.grin.mw link for general grin information.
CONTRIBUTING.md Outdated
* Further information and discussions are in the [Forum](https://forum.grin.mw), the [website](https://grin.mw), the [mailing list](https://lists.launchpad.net/mimblewimble/) and news channels like the [@grincouncil](https://twitter.com/grincouncil) and a (mostly unfiltered!) Twitter bot that collects headlines, mailing list posts, and reddit posts related to Mimblewimble/Grin: [@grinmw](https://twitter.com/grinmw)
* See `docs/*.md` and the folder structure explanations, and [the wiki](https://github.com/mimblewimble/docs/wiki).

* Also visit [Grin Docs and Resources]( https://docs.grin.mw/ ) for understanding and getting started Grin.
Copy link
Member

Choose a reason for hiding this comment

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

This sentence would be best if coupled with the one above which is also about documentation e.g.

Also see docs/*.md and the folder structure explanations, the wiki, and the official Grin documentation.

P.S. there are some spaces in the links which can be removed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Edited as you proposed, and removed space in the link.

@phyro phyro merged commit 03b007c into mimblewimble:master Jul 2, 2022
bayk added a commit to mwcproject/mwc-node that referenced this pull request Jun 21, 2024
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