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

Upgrade to Jandex 3.0.0 #1232

Merged
merged 1 commit into from
Sep 5, 2022
Merged

Upgrade to Jandex 3.0.0 #1232

merged 1 commit into from
Sep 5, 2022

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Aug 30, 2022

I noticed someone created the 2.3.x branch, but it was branched off of the 2.2.0 tag, which I believe is not right. So this PR, targeting the 2.3.x branch:

  1. fast-forwards the 2.3.x branch one commit after the 2.2.0 tag in the 2.2.x branch ([maven-release-plugin] prepare for next development iteration);
  2. fixes the <tag> element in the POM, as that should read HEAD outside of release commits;
  3. changes the <version> elements to say 2.3.0-SNAPSHOT;
  4. upgrades to Jandex 3.0.0.

With this PR merged to current 2.3.x (which points to a6d1ad5), I believe the 2.3.x branch would be in the correct state. Next, we would close #1227 (as we want to keep 2.2.x on Jandex 2.x). We could also close #1228, if 3.0.x is really automatically created from 2.2.x -- I believe we would actually also move 3.0.x to be based on 2.3.x.

Makes sense, or do I completely misunderstand how SmallRye OpenAPI is managed? Thank you!

@Ladicek Ladicek marked this pull request as ready for review August 30, 2022 10:40
@Ladicek
Copy link
Contributor Author

Ladicek commented Aug 30, 2022

Also, if you wanna take the first 3 commits from this PR and push them to 2.3.x directly, that would be fine with me :-) Or any other way you wish, of course.

@MikeEdgar
Copy link
Member

@Ladicek , this is good, thank you. Please go ahead and close the 3.0.x and 2.2.x PRs and I'll update the main PR and this with the changes for type references. A 3.0-RC is manually generated each time a release of 2.x occurs, with only the Jakarta changes applied with the to-jakarta.sh script. Once 3.0 is released that will stop, of course.

@Ladicek
Copy link
Contributor Author

Ladicek commented Aug 30, 2022

PRs to 2.2.x and 3.0x closed, thank you!

@sonarcloud
Copy link

sonarcloud bot commented Sep 5, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@MikeEdgar MikeEdgar added this to the 2.3.0 milestone Sep 5, 2022
@MikeEdgar MikeEdgar merged commit 1e09868 into smallrye:2.3.x Sep 5, 2022
@Ladicek Ladicek deleted the jandex3-2.3.x branch September 6, 2022 06:46
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.

2 participants