nx-guides icon indicating copy to clipboard operation
nx-guides copied to clipboard

Add a CONTRIBUTING.md to nx-guides

Open MridulS opened this issue 4 years ago • 9 comments

Mirror it with what we do in networkx/networkx

MridulS avatar Oct 07 '21 17:10 MridulS

Should we also add an explicit "Authors field" in the notebooks ? Can this be done via the sphinx theme?

MridulS avatar Oct 09 '21 13:10 MridulS

Should we also add an explicit "Authors field" in the notebooks ? Can this be done via the sphinx theme?

This can be done, but may introduce an extra layer of unnecessary complexity... I think the number and degree of contribution is more than adequately reflected in the git history, and it'd be very easy for an "author" field to fall out of sync with the actual contribution history for any given tutorial.

In all, I'm +1 for a nx-style contributors.md doc, and -1 on a per-file "authors" tracker.

rossbar avatar Oct 11 '21 18:10 rossbar

Hey @MridulS , I would like to do this. Should I create a PR?

inomag avatar Mar 09 '22 15:03 inomag

I know I said I was +1 in my previous comment, but honestly even that feels like overkill. GitHub already has a really nice interface for this info, so it seems silly to duplicate it. Maybe we can just link to the GitHub contribution page for nx-guides from the main page? Doing what NetworkX does (i.e. maintaining a contributor's list manually) seems not worth the effort here.

rossbar avatar Mar 11 '22 02:03 rossbar

Ah wait, I think I meant CONTRIBUTING.md like https://github.com/networkx/networkx/blob/main/CONTRIBUTING.rst 😅

Currently there is no information about how to submit the myst notebooks to the this repo.

MridulS avatar Mar 11 '22 06:03 MridulS

Got it. Thanks @MridulS

inomag avatar Mar 11 '22 09:03 inomag

@MridulS I am thinking of creating another CONTRIBUTING.md for this repository. But I have some queries -

  • Is the project only meant to be run on Ubuntu & MacOS platforms? Symlink is being used which doesn't work on Windows. Do we need to make some changes there for this project to work on Windows?
  • Earlier, after creating a pull request, I faced some issues in the CI workflows. Instead of repeatedly making changes in the code and waiting for the workflows to pass, should I include a guide on the workflows so the users can easily check the build process on their local systems?

Apart from that I will include the steps to change from jupyter notebook to markdown using jupytext, maintaining the correct file structure, etc

inomag avatar Apr 22 '22 18:04 inomag

@inomag you're right that we definitely need a contributor guide... it probably makes sense for one of us to take the first crack at it since we should (at least in principle) remember how we had structured the project :). I'll try to prioritize this and will definitely ping you to test drive the guide and ask you for feedback!

rossbar avatar Apr 26 '22 03:04 rossbar

@rossbar That would be great! I have done some part of it. https://gist.github.com/inomag/e3970148db2b8d07d4caf7a1231890c6

Though it is incomplete, can you give me suggestions what can I include further?

inomag avatar Apr 26 '22 09:04 inomag