Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add checkdeps #418

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Add checkdeps #418

merged 1 commit into from
Jan 17, 2024

Conversation

davidjharder
Copy link
Member

Add info about checkdeps to the following pages:

  • packaging changes
  • package.yml
  • packaging-practices (just a stub for now)

Fixes #417

@davidjharder davidjharder force-pushed the add-checkdeps branch 2 times, most recently from a0faa33 to 1deba72 Compare October 24, 2023 21:14
@GZGavinZhao
Copy link
Member

GZGavinZhao commented Oct 25, 2023

Thanks a lot @davidjharder! Everything looks good. I just need to give a little think on what wording I want to use for the "packaging practice" section. Would you mind if I take over/commandeer this PR from now on? Edit: you don't need to do anything. I just don't want to push before letting you know and then cause you to lose existing work you may have done locally.

@davidjharder
Copy link
Member Author

@GZGavinZhao I've committed all my work on this. Feel free to commandeer the PR.

@GZGavinZhao GZGavinZhao self-assigned this Oct 27, 2023
@TraceyC77
Copy link
Contributor

@GZGavinZhao - I'm changing this to Draft status since it looks like you want to add to it before it merges. If this is incorrect, my apologies.

@TraceyC77 TraceyC77 marked this pull request as draft October 30, 2023 19:01
@davidjharder
Copy link
Member Author

Hey @GZGavinZhao , do you mind if I merge this?

@davidjharder davidjharder marked this pull request as ready for review January 17, 2024 20:52
@GZGavinZhao
Copy link
Member

No problem, go ahead! Thanks for finishing this!

Add info about checkdeps to the following pages:

- packaging changes
- package.yml
- packaging-practices (just a stub for now)
@davidjharder davidjharder merged commit 57af840 into master Jan 17, 2024
1 check passed
@davidjharder davidjharder deleted the add-checkdeps branch January 17, 2024 21:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Packaging Docs: Update for checkdeps
3 participants