search

LSSTDESC / DC2-production

Configuration, production, validation specifications and tools for the DC2 Data Set.
BSD 3-Clause "New" or "Revised" License
11 stars 7 forks source link

Creating a release tag of this repo for Run2.2i #374

Closed heather999 closed 4 years ago

heather999 commented 4 years ago

It would be best to create a release tag for this repository for use during the Run2.2i DM DRP processing. As far as I know, all the required changes are in, though there are some outstanding PRs on this repo. Pinging @wmwv and @johannct I'm happy to go ahead and make a release tag, just want to make sure there are no pending updates.

heather999 commented 4 years ago

How do we get closure on this? I'd like to create a release tag for Run2.2i. Is there anything outstanding that we need? Unless I hear otherwise, pending some discussion tomorrow - I'll plan to create a tag, and we can always create another on if necessary. @wmwv @johannct

wmwv commented 4 years ago

I know of no impending updates to this repo.

wmwv commented 4 years ago

It's possible that we'd want to first merge https://github.com/LSSTDESC/DC2-production/pull/378 https://github.com/LSSTDESC/DC2-production/pull/379

But creating a tag and then creating another tag if necessary seems like a good reasonable way to make progress.

jchiang87 commented 4 years ago

It's possible that we'd want to first merge

378

379

I think we should just go ahead and merge those PRs.

wmwv commented 4 years ago

I've approved #378. It can be merged and then this release tagged.

379 would be a civilized thing to have but isn't directly relevant for Run 2.2i processing.

heather999 commented 4 years ago

Now that #378 has been merged. I went ahead and created a v0.4.0 tag following previous conventions. We can make follow on tags as needed. Thanks for the help. Closing this issue