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

DM-39212: Move no-instrument pipelines in cp_pipe to an "ingredients" directory #194

Merged
merged 2 commits into from May 25, 2023

Conversation

czwa
Copy link
Contributor

@czwa czwa commented May 15, 2023

Move pipeline defaults to ingredients directory.

@czwa czwa requested a review from TallJimbo May 16, 2023 20:35
@czwa czwa requested review from leeskelvin and removed request for TallJimbo May 22, 2023 22:03
@leeskelvin
Copy link
Contributor

Thanks @czwa, these changes look good. Would you be able to add a README similar to https://github.com/lsst/drp_pipe/blob/main/pipelines/README.md into the pipelines directory, and similarly another README into _ingredients similar to https://github.com/lsst/drp_pipe/blob/main/ingredients/README.md? (Those README's haven't been updated to reflect the new layout, but they should hopefully be useful). Otherwise, I think this is good, cheers!

@czwa czwa force-pushed the tickets/DM-39212 branch 2 times, most recently from 03fc911 to f10a504 Compare May 24, 2023 23:18
@czwa czwa merged commit 85bbc63 into main May 25, 2023
1 check passed
@czwa czwa deleted the tickets/DM-39212 branch May 25, 2023 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants