Skip to content

Add separate & improved how-to-contribute page - #210

Merged
dalito merged 7 commits into
mainfrom
issue152-improve-contrib-steps-docu
Nov 12, 2025
Merged

Add separate & improved how-to-contribute page#210
dalito merged 7 commits into
mainfrom
issue152-improve-contrib-steps-docu

Conversation

@dalito

@dalito dalito commented Nov 10, 2025

Copy link
Copy Markdown
Member

This pull request introduces several improvements to the documentation.

  • Completely rewrote how-to-contribute.md with expanded step-by-step instructions, diagrams, and tabbed UI guidance
  • Updated documentation build process to use uv package manager instead of pip
  • Standardized spelling of "Skosmos" (previously "SKOSMOS") and "ZB MED" throughout documentation
  • Added sphinxcontrib-mermaid extension for diagram support
  • Improved release process documentation with clearer formatting
  • Added multiple screenshots to guide contributors through GitHub web UI workflows

Open ToDos:

Closes #152

@dalito dalito changed the title Issue152 improve contrib steps docu Add separate & improved how-to-contribute page Nov 10, 2025
@dalito dalito moved this from New to In progress in Voc4Cat cross-repo view Nov 10, 2025
@dalito dalito added the documentation Improvements or additions to documentation label Nov 10, 2025
@dalito dalito self-assigned this Nov 10, 2025
@dalito
dalito force-pushed the issue152-improve-contrib-steps-docu branch from 6d1d7ca to 2da7a64 Compare November 12, 2025 22:57
@dalito
dalito marked this pull request as ready for review November 12, 2025 22:58
@dalito
dalito requested a review from a team as a code owner November 12, 2025 22:58
Copilot AI review requested due to automatic review settings November 12, 2025 22:58

This comment was marked as off-topic.

@dalito

dalito commented Nov 12, 2025

Copy link
Copy Markdown
Member Author

I moved the contribution guide out of the guidelines into the page https://nfdi4cat.github.io/voc4cat/docs_usage/how-to-contribute.html and extended it. At some point we may drop the Guidelines (V2.0) / All-in-one-page (as published) since it can slowly replaced by the update HTML-guidelines and the How-to-contribute guide in this PR.

@nmoust - I merge this PR now. Since the changes are big you can better review of the rendered HTML page than the markdown with MyST markup formatting.

@dalito
dalito merged commit 7123c58 into main Nov 12, 2025
7 checks passed
@github-project-automation github-project-automation Bot moved this from In progress to Completed in Voc4Cat cross-repo view Nov 12, 2025
@dalito
dalito deleted the issue152-improve-contrib-steps-docu branch November 12, 2025 23:14
@nmoust

nmoust commented Nov 14, 2025

Copy link
Copy Markdown
Collaborator

@dalito: Everything looks really nice! I would only add in the "Create the pull request" section, that if the user cannot see "base repository: nfdi4cat/voc4cat base: main" option they should click on the link above with the text: "compare across forks". This might be the case if the contributor checks the PR from their own fork. The "compare across forks" text is not visible in the screenshot.

@dalito

dalito commented Nov 14, 2025

Copy link
Copy Markdown
Member Author

@nmoust thanks for the feedback.

What you refer to, is a PR created from the "Pull Requests" tab? If the repo has differs from upstream it is possible to start the PR directly via the main "Code" tab (maybe this is not so clear?).

@nmoust

nmoust commented Nov 14, 2025

Copy link
Copy Markdown
Collaborator

From my fork of Voc4Cat (synched), when I submit an xlsx file in the inbox folder, and click "Propose changes" (screenshot 1), the next screen gives me the option to merge my "test-branch" to my main (screenshot 2). But if I click the "compare across forks" link, I get the option to merge my "test-branch" to the NFDI4Cat/Voc4Cat main (screenshot 3).

Screenshot0 Screenshot1 Screenshot2

@dalito

dalito commented Nov 14, 2025

Copy link
Copy Markdown
Member Author

I see. So it would be good to add a hint to click "compare across forks" if the user cannot select the upstream repo.

dalito added a commit that referenced this pull request Nov 16, 2025
* Improve how-to-contribute (styling and PR #210 follow-up)
* Clean-up README - remove duplicate contributing guide
* Make copilot PR reviews more precise & less noisy
* Avoid PR-from-org-warning for home-organization and switch to using gh-markdown alert messages
HendrikBorgelt pushed a commit to HendrikBorgelt/voc4cat that referenced this pull request Feb 13, 2026
* Improve how-to-contribute (styling and PR nfdi4cat#210 follow-up)
* Clean-up README - remove duplicate contributing guide
* Make copilot PR reviews more precise & less noisy
* Avoid PR-from-org-warning for home-organization and switch to using gh-markdown alert messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

📝Misunderstanding Step 7: Submit the updated Voc4Cat in a pull request

3 participants