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

Addition of worked example of GitHub #23

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

nk1g09
Copy link
Collaborator

@nk1g09 nk1g09 commented Aug 28, 2023

Adding an introductory guide to GitHub with a worked example showing contributing to the WFChemCookBook.

Please amend the location / contents as necessary, or let me know of any changes that need to be made.

Currently this guide is through the GUI on windows, but could be extended in future to include command line prompts, differences for Mac.
It also does not cover some more complex elements such as conflict resolution.

Copy link
Collaborator

@kroenlein kroenlein left a comment

Choose a reason for hiding this comment

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

@nk1g09 Great work! Really well written, nice screen shots.

We haven't properly documented the scope of the Cookbook (and our CONTRIBUTING documentation is pretty anemic). Had we done so, I'd say this piece is out of scope for our samples section, but it would be fantastic for augmenting our How To Contribute section. Do you have an opinion on any of the following?

  • Once I get a PR together baking out contributing, you could migrate this work into that piece of work
  • I could just directly include this work in my generated PR. I'd start from this branch, so you'd still get credit & blame.
  • We could try to identify a practical chemical information application of this example, and migrate to that. The explanations of Version Control etc. would be valuable as primary educational tools.

If this is going to be in the samples section, it'll need to conform to some templates that @stuchalk is putting together. Again, I apologize for pointing at a moving target on that.

@nk1g09
Copy link
Collaborator Author

nk1g09 commented Sep 28, 2023

Hi Ken, Apologies I didn't see this earlier (my github notfications clearly don't come through as expected). I'm happy for this to be changed as necessary, it probably is in the wrong section as it didn't really seem to fit with any of the existing items.

Happy for you to adapt this for the contributing guide. This was angled as an introduction to Github, and the contribution use case was identified as a good one as we didn't have an obvious chemistry angle. But we could definitely split it out if we have a different exemplar that I could redo the screenshots for.

@stuchalk
Copy link
Member

NIki, I have just added documentation about contributing to the Cookbook on the repository's Wiki. If you have time would appreciate a quick read and feedback... If possible would be good to touch base on getting your contribution finished before the holidays...

@stuchalk stuchalk added the documentation Improvements or additions to documentation label Dec 12, 2023
@stuchalk stuchalk added this to the Version 0.5 milestone Dec 12, 2023
@stuchalk
Copy link
Member

@nk1g09 Please go ahead and modify you recipe to support the PR via a fork workflow. I will get it in the Cookbook as soon as I can ...

@stuchalk stuchalk added the contribution A new proposed addition to the cookbook label Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution A new proposed addition to the cookbook documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants