Skip to content

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

melissawm
Copy link
Member

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:

  • Highlighting the common themes to all workflows first
  • Including more signposting
  • Using dropdowns for the specialized build - I think that section will need a more careful look tomorrow
  • Grouping similar themes together so the information is less scattered.
    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.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label May 21, 2025
@psobolewskiPhD
Copy link
Member

Interesting, so Circle isn't running at all anymore? Gah!

@psobolewskiPhD psobolewskiPhD added this to the 0.6.2 milestone May 22, 2025
@psobolewskiPhD
Copy link
Member

@napari-bot make slimfast

@psobolewskiPhD
Copy link
Member

Very wierd. The comment triggered a run, but it's marked as triggered by Juan and from main:
image
But pr #713 worked correctly?
image


````{dropdown} napari gallery of examples only
Copy link
Member

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:

Suggested change
````{dropdown} napari gallery of examples only
````{dropdown} napari/docs and napari gallery of examples


````{dropdown} Single example in the gallery
Copy link
Member

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.

Suggested change
````{dropdown} Single example in the gallery
````{dropdown} napari/docs and a single example in the gallery

Copy link
Member

@psobolewskiPhD psobolewskiPhD left a 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.

Copy link
Contributor

@willingc willingc left a 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.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Before you start
## Ask for guidance

Comment on lines +24 to +29
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.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
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.

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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants