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

Release 0.21.0.CR1 #1014

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Release 0.21.0.CR1 #1014

merged 1 commit into from
Oct 28, 2024

Conversation

geoand
Copy link
Collaborator

@geoand geoand commented Oct 28, 2024

No description provided.

@geoand geoand requested a review from a team as a code owner October 28, 2024 09:16
@geoand geoand merged commit 58f1439 into main Oct 28, 2024
2 checks passed
@geoand geoand deleted the release-0.21.0.CR1 branch October 28, 2024 09:16
@geoand
Copy link
Collaborator Author

geoand commented Oct 28, 2024

@gastaldi I tried to do a release with the new process but it failed: https://github.com/quarkiverse/quarkus-langchain4j/actions/runs/11550856264/job/32146686766

Any idea what the problem is? It seems to not like the fact that our samples have fixed versions - but that is done on purpose and those versions are updated automatically when the release is out

@gastaldi
Copy link
Member

That is really strange, it failed in the release:prepare, and that step hasn't changed from the original process. Do you get the same error if you run it locally?

@geoand
Copy link
Collaborator Author

geoand commented Oct 28, 2024

Indeed it does. Weird, we never encountered that in previous releases

@geoand
Copy link
Collaborator Author

geoand commented Oct 28, 2024

Aha, it seems that we used to do -Darguments=-DperformRelease -DperformRelease which meant that the samples were disabled.

I can just add that back to the two releases executions. I don't see how it could cause problems with the new release workflows, do you?

@geoand
Copy link
Collaborator Author

geoand commented Oct 28, 2024

Ah, I can't add those now, can I?

@gastaldi
Copy link
Member

Using performRelease in the release:prepare looks wrong IMHO. This is what it does: https://github.com/quarkiverse/.github/blob/main/.github/workflows/prepare-release.yml#L117

@geoand
Copy link
Collaborator Author

geoand commented Oct 28, 2024

Okay, I just need a way to deactivate the samples profile when a release is done and using:

      <activation>
        <property>
          <name>performRelease</name>
          <value>!true</value>
        </property>
      </activation>

was the easiest way.

What do you propose that be changed to?

@gastaldi
Copy link
Member

I can think of a few options:

@geoand
Copy link
Collaborator Author

geoand commented Oct 28, 2024

Thanks!

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