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

3.30.20 release checklist #352

Closed
18 tasks done
k-doering-NOAA opened this issue Jul 18, 2022 · 20 comments
Closed
18 tasks done

3.30.20 release checklist #352

k-doering-NOAA opened this issue Jul 18, 2022 · 20 comments
Assignees
Labels
workflows related to workflows/gh-actions
Milestone

Comments

@k-doering-NOAA
Copy link
Contributor

k-doering-NOAA commented Jul 18, 2022

It's not yet time for the 3.30.20 release (we are waiting on a final version of ADMB 13), but feel free to add notes about the release here

Checklist for before prerelease

  • Update GHA to use ADMB 13 (IT)
  • SS testing and debugging completed (IT)
  • r4ss updated (IT)
  • Put together pre-release announcement (RM)
  • code committed and tagged in repo as v3.30.20-prerel, which will trigger a gha to build the prerelease executables (KJ)
  • All exes added to github releases as v3.30.20-prerel (KJ) (use exe artifacts from the gha in step above)
  • Announce prerelease (RM)

Checklist for before release

Checklist for after release

  • examples updated in user-examples repo using this script (KJ)
  • Update test-models reference files to the release version using this update script (EG). also tag after updating with release number.
  • Removed "resolved" tag and close all issues worked in the milestone for this release (RM)
  • Move unworked issues for the release milestone to the next milestone (RM)
@k-doering-NOAA
Copy link
Contributor Author

@kellijohnson-NOAA has kindly agreed to coordinate this release, so she will be making sure the checklist gets completed when the time comes for the release.

@k-doering-NOAA
Copy link
Contributor Author

Note I reassigned tasks that I usually do to coordinate a release (by changing initials in the checklist), so please take a look and let me know if you have any concerns about any new tasks! I will be available to answer questions if they arise.

@Rick-Methot-NOAA
Copy link
Collaborator

Rick-Methot-NOAA commented Sep 15, 2022

The change log was still a blank slate for 3.30.20. So rather than create al those entries, I am trying to just improve the labelling and linking for all the PRs and Issues in Milestone 3.30.20. Is a milestone view publicly accessible?

We can improve this approach by:

  • assuring that every PR is linked to its originating issue
  • adding Milestone designation to each PR
  • adding at least one Label to each PR (like selectivity)

@kellijohnson-NOAA
Copy link
Contributor

If you navigate to issues you can enter

milestone:3.30.20 

in the search bar and it will search for issues and pull requests with that milestone.

@iantaylor-NOAA
Copy link
Contributor

This is an area where Kathryn was the key to making the changelog happen.

Kelli's search could be linked in a release message but the list is pretty messy: https://github.com/nmfs-stock-synthesis/stock-synthesis/issues?q=milestone%3A3.30.20+

I was just looking at github projects and see that they are private by default. For instance, this milestone tracker is not visible when I'm not logged into github https://github.com/orgs/nmfs-stock-synthesis/projects/2/views/1, but it could easily be made public. It's a cleaner list for folks to look at, but still not great.

If you want a change log like what we've done in the past, I can budget some time tomorrow or next week to try to populate one.

@Rick-Methot-NOAA
Copy link
Collaborator

Rick-Methot-NOAA commented Sep 16, 2022

Thanks for feedback. I know how to do it myself, what I am looking for is something that any user could easily do. Given that none of the issues and PR were documented in the change log yet, I think it better to find a way to do it with github tools. I agree that a project table will be cleaner looking that the milestone list and that probably is the way to go. But it requires that all items get labelled so they can be automatically displayed in the project table.

@k-doering-NOAA
Copy link
Contributor Author

I thought about this a bit in the past and couldn't come up with a good, concise way to use the project board. What I had hoped to do (and didn't have time for) was to create a ghaction that pulled info from the Pull request form and populated the change log for us. Unfortunately, I think it is hard to automate a change log, because issues (and even pull requests) tend to be a bit messy.

At least with the PR template, I think someone could copy/paste the lines from it for this release, which is a little less work than in the past.

@Rick-Methot-NOAA
Copy link
Collaborator

Rick-Methot-NOAA commented Sep 16, 2022

I find this view from current Excel change log:
image

no better than this view from modified project board:
image

The problem is sharing that view with the community. It is a convenient format for reviewing what has been done; I can even edit labels from within that view.

@iantaylor-NOAA
Copy link
Contributor

Good point Rick
To that end, I just created a new project called "change log", added all issues matching the following:
label:"change log" milestone:"3.30.19" status:"Done"
and changed the settings to make it public.
Here's the link https://github.com/orgs/nmfs-stock-synthesis/projects/11/views/1 which I think we could put into the release notes. Would that work?

Initially I created a "public change log" view within the "milestone tracker" project (it's still there), but realized that it would require making all the views in that project public if we want to make one of them public. I'm not sure that's a bad thing, but it could create more confusion than a single-view project like the one linked above.

@Rick-Methot-NOAA
Copy link
Collaborator

Thanks Ian. I had created a change log view within the ss3-issue-tracker project, but if we are to share this it makes sense to have a dedicated project just for that purpose.

But we do not want too many projects because items need to be individually added to each project, so that favors putting things into just one project from which we can share views.

The project system in github favors putting Issues into the project and then letting the system display linked PR. I like that. So, I should stop putting PR directly into the project because it clutters things by having something represented by both a PR and an Issue, and not even adjacent rows.

@iantaylor-NOAA
Copy link
Contributor

I think we could train ourselves to apply the "change log" label and add to the "change log" project at the same time on an issue worth including. Or we could do away with the "change log" label and just use the project board. Presumably those issues would already have been added to the "ss3-issue-tracker" project at the point that they started getting tracked or worked on.

I added the 3.30.20 milestone issues to the change log project and removed one pull request that were in that project. But that PR (#270) maybe should belong anyway because it includes a partial fix for one issue (#265) and the PR isn't showing up as linked to either #265 or #268 (which is now in the project).

I don't know how you created the milestone groupings in the ss3-issue-tracker.

@Rick-Methot-NOAA
Copy link
Collaborator

Rick-Methot-NOAA commented Sep 16, 2022

I just rearranged the columns so you can see each entries milestone level. It is easy to click that entry pulldown to change it to a different milestone assignment. Similarly, easy to add/delete labels.

I think we should be using the workflow Label more for things like gha changes. That way we can use -Label in the Project table to not display those items if we choose.

@iantaylor-NOAA
Copy link
Contributor

Thanks for the tip on the grouping option. I'm now sold on using a project board to replace the change log.
Yes, this can be a work in progress and we can get feedback from users and refine as we go. I'm not sure how many of them actually read the change log anyway.

@k-doering-NOAA
Copy link
Contributor Author

Yes, great work, Rick! It's looking good.

@Rick-Methot-NOAA
Copy link
Collaborator

@kellijohnson-NOAA
@iantaylor-NOAA
@nschindler-noaa
What is status of work to get ready for this release? I would like to get it out on Friday this week.

@kellijohnson-NOAA
Copy link
Contributor

@Rick-Methot-NOAA given that we are not doing a prerelease can I cross off the last four items in the pre-release checklist?

@Rick-Methot-NOAA
Copy link
Collaborator

go for it.

@kellijohnson-NOAA
Copy link
Contributor

The repo has been tagged with v.3.30.20 and the executables can be found in this zip file. The files can also be found on the google drive here. I tested the windows executable with {ss3sim} and everything appears to be fine. @Rick-Methot-NOAA do you want me to "Draft a new release", which is a button available here and add the exes or do you want to do that?

@Rick-Methot-NOAA
Copy link
Collaborator

You're on a roll. Please do draft that new release and link to the release message that I will broadcast via VLAB

@Rick-Methot-NOAA
Copy link
Collaborator

Thank you Kelli and everyone contributing to this release

@Rick-Methot-NOAA Rick-Methot-NOAA removed the in progress This is being worked on in a branch label Oct 26, 2022
e-perl-NOAA added a commit that referenced this issue Dec 27, 2022
add links and further detail based on the 3.30.20 release (#352)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
workflows related to workflows/gh-actions
Projects
Status: Done
Development

No branches or pull requests

6 participants