-
Notifications
You must be signed in to change notification settings - Fork 41
Update contributing docs page #715
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
base: main
Are you sure you want to change the base?
Conversation
Interesting, so Circle isn't running at all anymore? Gah! |
@napari-bot make slimfast |
Very wierd. The comment triggered a run, but it's marked as triggered by Juan and from main: |
|
||
````{dropdown} napari gallery of examples only |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this one isn't strictly true, the docs will be built, just no notebook cells run. Not sure how best to communicate that.
Based on the first two, I I guess I suggest:
````{dropdown} napari gallery of examples only | |
````{dropdown} napari/docs and napari gallery of examples |
|
||
````{dropdown} Single example in the gallery |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on the logic above, but I dont love it.
````{dropdown} Single example in the gallery | |
````{dropdown} napari/docs and a single example in the gallery |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really great!
I left a nitpick and some confused comments, but I think even as is merging this is a major plus versus the current state.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few suggestions. Much improved @melissawm
@@ -1,10 +1,34 @@ | |||
(contributing-docs)= | |||
# Contributing Documentation | |||
|
|||
This guide will teach you how to submit new documents to napari's usage | |||
documentation. | |||
This guide will teach you how to submit your contributions to napari's documentation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This guide will teach you how to submit your contributions to napari's documentation. | |
This guide will teach you how to contribute to napari's documentation. |
|
||
## Organization of the documentation | ||
In general, contributing requires: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general, contributing requires: | |
To begin contributing, you will need: |
- Some familiarity with [`git`](https://git-scm.com). | ||
- A [GitHub](https://github.com) account. | ||
|
||
There are two ways you can approach your contribution: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are two ways you can approach your contribution: | |
Choose an approach to contributing: |
changes _before_ submitting them. This approach requires a local setup of the | ||
napari and docs repositories. | ||
|
||
## Before you start |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
## Before you start | |
## Ask for guidance |
If you'd like to contribute a brand new document to our usage section, it might | ||
be worth [opening an issue](https://github.com/napari/docs/issues/new/choose) | ||
on our repository first to discuss the content you'd like to see and get some | ||
early feedback from the community. The napari team can also suggest what type of | ||
document would be best suited, and whether there are already existing documents | ||
that could be expanded to include the content you think is lacking. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you'd like to contribute a brand new document to our usage section, it might | |
be worth [opening an issue](https://github.com/napari/docs/issues/new/choose) | |
on our repository first to discuss the content you'd like to see and get some | |
early feedback from the community. The napari team can also suggest what type of | |
document would be best suited, and whether there are already existing documents | |
that could be expanded to include the content you think is lacking. | |
If you'd like to contribute a brand new document to our usage section, it's | |
worth [opening an issue](https://github.com/napari/docs/issues/new/choose) | |
on our repository to discuss the content you'd like to see and get some | |
early feedback from the community. The napari team can also suggest what type of | |
document would be best suited, and whether there are already existing documents | |
that could be expanded to include your proposed content. |
References and relevant issues
--
Description
This is a suggestion only, and I took a heavy axe to this document 😅 But I just wanted to experiment with a different organization of things. I tried:
This can obviously be very subjective so I'm definitely happy to address feedback and keep working on it. In particular I'll come back tomorrow to review the links and other internal references since they might have changed.
The main content is still all there, I just moved things around and maybe reworded things here and there.