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

Built on @dtekinoglu's contributing guide: added format guidelines and slightly modified brief roadmap #99

Merged
merged 13 commits into from
Nov 7, 2023

Conversation

20kavishs
Copy link
Contributor

Built on discussion at #98

  1. Added Format Guidelines

  2. Modified the Brief Roadmap to discuss contributions to /exploratory-notebooks or /generators as well.

@20kavishs 20kavishs mentioned this pull request Mar 26, 2023
@MridulS MridulS mentioned this pull request Mar 31, 2023
@MridulS
Copy link
Member

MridulS commented Mar 31, 2023

Thanks for picking this up, I pushed some changes to fix up the CI so you would need to do a git pull before you push anything on top of this. The CI passes now and you can look at the output at circle CI to see how this guide would look on the website. https://output.circle-artifacts.com/output/job/32e0c8dd-d042-4c06-aa8b-1191c8999891/artifacts/0/site/_build/html/content/contributing.html

@20kavishs
Copy link
Contributor Author

Awesome thanks!

content/contributing.md Outdated Show resolved Hide resolved
@dschult dschult force-pushed the addFormatGuidelines branch from 5129584 to f599659 Compare August 31, 2023 23:52
Copy link
Member

@dschult dschult left a comment

Choose a reason for hiding this comment

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

I think this is ready for merging and we can update it as needed. It's a good start anyway.

@MridulS MridulS merged commit bf9bbab into networkx:main Nov 7, 2023
0 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants