Skip to content

Add version support information table to docs. #7833

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
Feb 24, 2022

Conversation

stuartarchibald
Copy link
Contributor

This adds a table to the docs that shows which versions of Numba's
dependencies were tested and known to work against which version of
Numba. It also updates the first_rc_checklist.md issue template
to ensure that this information is provided along side future
releases.

This adds a table to the docs that shows which versions of Numba's
dependencies were tested and known to work against which version of
Numba. It also updates the `first_rc_checklist.md` issue template
to ensure that this information is provided along side future
releases.
@@ -0,0 +1,37 @@
.. _support_info:
Copy link
Member

Choose a reason for hiding this comment

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

I think this page should be referenced in the installation doc: https://numba--7833.org.readthedocs.build/en/7833/user/installing.html

Copy link
Member

Choose a reason for hiding this comment

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

I would actual raise the argument, that the version support information table, should go alongside our canonical reference for the other dependencies, here:

https://numba.readthedocs.io/en/stable/user/installing.html#dependency-list

This would mean, all dependency information is in a single place and can percolate into the various places (setup.py et.c..) from there. How does that sound?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

867223f makes this move, I'm happy either way, am just glad this reference now exists!

@sklam sklam added 4 - Waiting on author Waiting for author to respond to review and removed 3 - Ready for Review labels Feb 17, 2022
@esc esc mentioned this pull request Feb 17, 2022
@stuartarchibald stuartarchibald added 4 - Waiting on reviewer Waiting for reviewer to respond to author and removed 4 - Waiting on author Waiting for author to respond to review labels Feb 22, 2022
Copy link
Member

@sklam sklam left a comment

Choose a reason for hiding this comment

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

LGTM now

@sklam sklam added 5 - Ready to merge Review and testing done, is ready to merge and removed 4 - Waiting on reviewer Waiting for reviewer to respond to author labels Feb 22, 2022
Copy link
Member

@esc esc left a comment

Choose a reason for hiding this comment

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

thank you!

@sklam sklam merged commit 208aa72 into numba:main Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Ready to merge Review and testing done, is ready to merge Effort - medium Medium size effort needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants