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

Describe pull request workflow #1104

Closed
1 task
l-emele opened this issue Apr 6, 2022 · 8 comments
Closed
1 task

Describe pull request workflow #1104

l-emele opened this issue Apr 6, 2022 · 8 comments
Labels
organisation Concerning the repository and GitHub related tasks stale already discussed issues that haven't got worked on for a while

Comments

@l-emele
Copy link
Contributor

l-emele commented Apr 6, 2022

Description of the issue

Describe somewhere (either in CONTRIBUTING.md or the wiki) the workflow within a pull request.

Ideas of solution

Topics:

  • Draft vs ready for review
  • Assigning reviewers
  • What should reviewer check?
    • Right classification and axioms
    • Spelling and grammar of definitions
    • CHANGELOG.md
    • Term tracker items
    • ...

Workflow checklist

  • I am aware of the workflow for this repository
@l-emele l-emele added organisation Concerning the repository and GitHub related tasks To do Issues that haven't got discussed yet labels Apr 6, 2022
@markus-rothkoetter
Copy link
Contributor

additional topic suggestions:

  • What do the automated checks do?
  • Is protégé configured for numerical identifers?
  • usage of automatic closing for issues

@l-emele
Copy link
Contributor Author

l-emele commented Apr 7, 2022

@markus-rothkoetter
Copy link
Contributor

Once the wiki-entry is finished:
Shall we exchange the "Review"-section in CONTRIBUTING.md for a brief description ("Our PR-workflow is special due to term tracker ... ") and a link to the wiki?

This would ...

  • .. decrease maintenance effort (divide-and-conquer)
  • ... ease the onboarding of new users as they aren't overwhelmed by a super-sized CONTRIBUTING.md (overview-first, details-on-demand)

@github-actions github-actions bot removed the To do Issues that haven't got discussed yet label Apr 7, 2022
@l-emele
Copy link
Contributor Author

l-emele commented Apr 7, 2022

That is a good idea, the CONTRIBUTING.md is very long.

@markus-rothkoetter
Copy link
Contributor

I did a first draft here: https://github.com/OpenEnergyPlatform/ontology/wiki/Pull-request-workflow

I adjusted the wiki sidebar-link to lead to that page the old one hit a non-existent page ("-workflow" was missing in the link)
Additionally, I think it's good to a a small indicator to show if a wiki page is work in progress. (like 🚧 in the sidebar-entry)

I haven't been able to clone the wiki locally, so shall I work on the live-version of the wiki when I'm extending your draft @l-emele ?

@l-emele
Copy link
Contributor Author

l-emele commented Apr 7, 2022

I adjusted the wiki sidebar-link to lead to that page the old one hit a non-existent page ("-workflow" was missing in the link) Additionally, I think it's good to a a small indicator to show if a wiki page is work in progress. (like 🚧 in the sidebar-entry)

Basically everything in the wiki is work in progress...

I haven't been able to clone the wiki locally, so shall I work on the live-version of the wiki when I'm extending your draft @l-emele ?

That is fine. Git pushes and live edits are here basically the same, as there aren't pull requests or something similar for the wiki.

@markus-rothkoetter
Copy link
Contributor

Basically everything in the wiki is work in progress...

I created a new issue #1106 in order to tackle this step-by-step.

That is fine. Git pushes and live edits are here basically the same, as there aren't pull requests or something similar for the wiki.

Good to know. Then I'll add further refinements to the page later.
I think to incorporate the content from the section in CONTRIBUTING.md is a good starting point.

@stale stale bot added the stale already discussed issues that haven't got worked on for a while label Apr 24, 2022
@stap-m stap-m added this to the oeo-release-1.11.0 milestone Apr 25, 2022
@markus-rothkoetter
Copy link
Contributor

Closing this as the wiki-page 📚 is at 90 % completion.

Only references to other pages and minor content additions are missing.
I'll monitor these in the wiki-toplevel issue #1106

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
organisation Concerning the repository and GitHub related tasks stale already discussed issues that haven't got worked on for a while
Projects
None yet
Development

No branches or pull requests

3 participants