Skip to content

Add short guide on updating constraints files #460

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

Merged
merged 2 commits into from
Aug 29, 2024

Conversation

melissawm
Copy link
Member

References and relevant issues

N/A

Description

Adds a short guide on updating the constraints files for core developers. I wonder if we should automate this with a bot or action (expanding the update test constraints action perhaps?)

@Czaki
Copy link
Contributor

Czaki commented Jul 19, 2024

@melissawm
Copy link
Member Author

But that is for tests only, right?

@Czaki
Copy link
Contributor

Czaki commented Jul 20, 2024

But that is for tests only, right?

No, it generates PRs like this napari/napari#7045
If you see the list of modified files, you will see that docs constraints are also updated.

jni pushed a commit to napari/napari that referenced this pull request Jul 22, 2024
)

# References and relevant issues
Follow-up to napari-sphinx-theme release 0.5.0.

# Description
Updates docs constraints file for python 3.10, also adds a constraints
file for docs on python 3.11.

I am not sure if I'm doing things right, but would love to learn. I am
also submitting a docs PR to detail this process in the core developers
guide.

UPDATE: napari/docs#460

Co-authored-by: Peter Sobolewski <[email protected]>
@jni
Copy link
Member

jni commented Aug 1, 2024

Should we also add the bot usage here, and @Czaki does the bot work in the docs repo?

@melissawm
Copy link
Member Author

Hi folks, not sure what the next step is here. Maybe we should close since the action runs automatically?

Czaki
Czaki previously requested changes Aug 16, 2024
Copy link
Contributor

@Czaki Czaki left a comment

Choose a reason for hiding this comment

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

I have added a few suggestions with information that should be added, but I think that not using my wording. If you agree with me, feel free to write it the own way.

@melissawm
Copy link
Member Author

Updated, thanks @Czaki !

@Czaki Czaki dismissed their stale review August 16, 2024 13:24

will try to find time later but not want to block

@psobolewskiPhD
Copy link
Member

Thanks @melissawm ❤️

@melissawm melissawm added this to the 0.5.3 milestone Aug 22, 2024
@melissawm
Copy link
Member Author

Hi friends, any last comments?

@jni jni merged commit d45b826 into napari:main Aug 29, 2024
8 checks passed
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
Development

Successfully merging this pull request may close these issues.

4 participants