Skip to content
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

Fixing Table of Contents/ Nav #266

Merged
merged 1 commit into from
May 31, 2024
Merged

Fixing Table of Contents/ Nav #266

merged 1 commit into from
May 31, 2024

Conversation

BSuperbad
Copy link
Contributor

It was slightly confusing to have 'Migrate setup.py to a pyproject.toml' (for existing packages) in the getting started/ tutorial setup portion of the table of contents/ nav. Proposing to move it down under a new heading 'Hatch for Existing Packages'

Thank you!

Copy link
Member

@lwasser lwasser left a comment

Choose a reason for hiding this comment

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

@BSuperbad thank you for this contribution! once CI runs i'll get it merged! it was great meeting you at pyconus this year!

@lwasser
Copy link
Member

lwasser commented May 30, 2024

@all-contributors please add @BSuperbad for code, review

Copy link
Contributor

@lwasser

I've put up a pull request to add @BSuperbad! 🎉

@lwasser lwasser merged commit adb5307 into pyOpenSci:main May 31, 2024
4 checks passed
@lwasser
Copy link
Member

lwasser commented Jun 6, 2024

this addresses #264

@BSuperbad THANK YOU for this submission! one small tip - when you open an pr - at the top you can reference the issue that you opened like this

closes #264

this will magically then close the issue once the pr is merged!! best of luck to you in your program that you're working on! it was fun to meet you at pycon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

2 participants