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

Preperations for acknowledging via allcontributors bot #70

Merged
merged 4 commits into from
Jun 26, 2020

Conversation

adswa
Copy link
Collaborator

@adswa adswa commented Jun 26, 2020

This PR

  • Moves README.rst into docs/source and adds a mark-down based README file in its place. This is needed because the allcontributors-bot requires a README.md. I have copied some of the README.rst contents, but felt that given that it seems to be the intro of the open-brain-consent docs, not all of its contents are required for the README of the GitHub page. I've kept a general intro and put in additional pointers to the docs and contributing.
  • Adds a short contributing section to the docs' source that mentions the allcontributors bot.

Once this PR (or a similar PR that creates a README.md file) is merged, we can start using the bot. Note that it is possible to collect the contributors information in a file other than the README, so speak up if you disagree with putting contributions to the "frontpage" - IMHO it is beneficial to recognize contributions very visibly. :)

ping @yarikoptic

This is because the allcontributors bot only works with markdown-based README files

As the rst README is more of an open-brain-consent intro than a GitHub repo README
I'll move it into the source of the docs and will create a dedicated repo README.md
The main reason for this change is that it allows to use the allcontributors bot, which
requires a markdown based README file
@yarikoptic
Copy link
Member

THIS IS AWESOME! Thank you so much @adswa !

@yarikoptic yarikoptic merged commit 3e80b9a into master Jun 26, 2020
@adswa adswa deleted the nf-acknowledgement branch June 26, 2020 21:22
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

2 participants