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

version number in spec files is not correct for 1.5.x #783

Closed
schristley opened this issue Apr 12, 2024 · 20 comments
Closed

version number in spec files is not correct for 1.5.x #783

schristley opened this issue Apr 12, 2024 · 20 comments
Assignees

Comments

@schristley
Copy link
Member

Both of the AIRR schema spec files say "1.4" when should be "1.5".

Because the master branch has already moved significantly towards 2.0, I've created a release-1.5 branch starting from the v1.5.1 tag according to our release processes. This branch won't get merged to master but will stay in case minor fixes need to be done for the 1.5.x release chain.

@schristley
Copy link
Member Author

@javh If the release-1.5 branch looks ok to you, and nothing else is missing, we should tag it with v1.5.2 and push a release.

@schristley
Copy link
Member Author

@bcorrie My understanding is that CRWG decided not to move the ADC API forward to v1.5.x? Is that true, or should we also update the ADC API specs to point to v1.5.2 and include in the release?

@javh
Copy link
Contributor

javh commented Apr 15, 2024

@schristley, this looks good to me. Technically, there is a change to the specs so we should release new versions of the R and python packages to CRAN and PyPI. But, it's so minor I'm kind of inclined to skip that step and just do a GitHub release only.

Thoughts?

@schristley
Copy link
Member Author

@javh I think that's ok. For me it's the GitHub tag/release and version number correction that's most important. I suppose we can always do the R and python packages later if people request it.

@javh
Copy link
Contributor

javh commented Apr 16, 2024

Sounds good to me. I'll make the draft release and email the working group to confirm.

@javh
Copy link
Contributor

javh commented Apr 21, 2024

I was making the draft release (in https://github.com/airr-community/airr-standards/releases) and I think we may have a problem with our release tags. v1.5.1, v1.4.2, and v1.4.1-beta don't seem to be actual releases, but instead tags that snuck in with PRs.

Meaning, the release-1.5 maintenance branch should start from the v1.5.0 tag and we should remove those extra tags from the repo.

Can you take a look at confirm? If I'm correct, it's at least easy to fix.

@schristley
Copy link
Member Author

Yeah, those tags look odd, as in they don't have a release associated with them. v1.4.2 is suspect as it's between v1.5.0 and v1.5.1. and v1.4.1-beta doesn't sound like a real release either. I guess you are saying that v1.5.1 never happened either, makes sense as I don't see any v1.5.1 release notes.

Yeah, looking at the commits between v1.5.0 and v1.5.1, there is a bunch of v2.0 stuff in there, we don't want that. You are right, should delete the release-1.5 branch and re-branch it off of v1.5.0.

@javh
Copy link
Contributor

javh commented Apr 21, 2024

Sounds good. We'll need to nuke all those rogue tags too.

@schristley
Copy link
Member Author

Ok, regenerated release-1.5 branch and removed those rogue tags. Also setup branch protections and checks.

I have a version of the airr-js package because I need to use it with v1.5 until v2.0 is released and I'm able to move it.

@schristley
Copy link
Member Author

@javh skip v1.5.1 and leave as v1.5.2, or change your draft release?

@javh
Copy link
Contributor

javh commented Apr 30, 2024

I'll change the draft release.

@javh
Copy link
Contributor

javh commented May 6, 2024

I looked through the branch and it looks ready to go to me except for:

  • Top level release notes
  • Some harmonization of version numbers

I'll take care of those after the wg call today and do the release.

But, one thing... Adding the js package seems much bigger than a patch release. I don't have an objection, but what do you think about making the release v1.6.0 (still off the release-1.5 branch)?

@schristley
Copy link
Member Author

schristley commented May 6, 2024 via email

@schristley
Copy link
Member Author

@javh If you want to pedantic about it, even though the js package seems to add a bunch, it's only a patch release.

@javh
Copy link
Contributor

javh commented May 14, 2024

I think we might want to add a python release to this with the versioneer fix (#744).

@schristley
Copy link
Member Author

@javh I think you must have those bogus tags in your local git because when you pushed your commits on Wednesday, those tags came back

@schristley
Copy link
Member Author

I only know this because of the messages posted to the slack channel, otherwise I don't know how to see tag history through the GitHub gui.

12:52
Tag created by javh
v1.5.1
https://github.com/[airr-community/airr-standards](https://github.com/airr-community/airr-standards)|airr-community/airr-standardsairr-community/airr-standards

@javh
Copy link
Contributor

javh commented May 17, 2024

Crap. I'll fix it. Thanks.

@javh
Copy link
Contributor

javh commented May 21, 2024

Okay, I've copied over the versioneer changes, decided on v1.5.1, updated the news, and done a basic install test on the python library. The branch should be ready to go.

I still need to sanity check the docs before publishing the release, but I'm about to get on a plane. :)

I'll try to wrap this up as soon as I get a chance.

@javh
Copy link
Contributor

javh commented Jun 2, 2024

v1.5.1 release is done and I pushed the python package to PyPI. I did not update the R package in CRAN, as there were no functional changes.

@javh javh closed this as completed Jun 2, 2024
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

No branches or pull requests

2 participants