Remove Duplicate CONTRIBUTING.md and add CHANGELOG to docs#381
Merged
martinRenou merged 12 commits intogeojupyter:mainfrom Jan 22, 2025
Merged
Remove Duplicate CONTRIBUTING.md and add CHANGELOG to docs#381martinRenou merged 12 commits intogeojupyter:mainfrom
martinRenou merged 12 commits intogeojupyter:mainfrom
Conversation
Contributor
|
|
arjxn-py
force-pushed
the
duplicate-contributing
branch
from
0af72a0 to
b72ad20
Compare
Contributor
|
Integration tests report: appsharing.space |
Contributor
|
Integration tests report: appsharing.space |
arjxn-py
force-pushed
the
duplicate-contributing
branch
from
47b1009 to
90153e4
Compare
Member
Author
martinRenou
reviewed
Jan 22, 2025
martinRenou
changed the title
mfisher87
reviewed
Jan 22, 2025
Member
There was a problem hiding this comment.
Maybe this should be in the docs directory so we can use sphinx directives and plugins to render a richer user experience. Perhaps instead of including from the repo root into the docs, we should hyperlink from the repo root to the docs?
Member
Author
There was a problem hiding this comment.
Sure, that sounds good to me.
Apologies as this may have been discussed a little more before going this way.
Member
There was a problem hiding this comment.
No sweat at all! We can't discuss everything in advance, and especially with the team being globally distributed, EAFP! ;)
HaudinFlorence
pushed a commit
to HaudinFlorence/jupytergis
that referenced
this pull request
Jan 28, 2026
…r#381) * update contributing.md accordingly as contributing.rst * try using myst-parser to parse md * Install myst-parser via pip * typo * pin myst-parser * try building RTD * lint * pass pre-commit * fic CI * Add changelog to docs too * fix rtd build
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Shall close #380
Checklist
Resolves #XXX.Failing lint checks can be resolved with:
pre-commit run --all-filesjlpm run lint📚 Documentation preview: https://jupytergis--381.org.readthedocs.build/en/381/
💡 JupyterLite preview: https://jupytergis--381.org.readthedocs.build/en/381/lite