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

Fix API compatibility workflow failing when building previous version #2694

Conversation

Marcono1234
Copy link
Collaborator

@Marcono1234 Marcono1234 commented Jun 1, 2024

Purpose

Fixes the current API compatibility workflows failures

Description

The workflow is currently failing for the jpms-test module:

Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.13.0:compile (default-compile) on project jpms-test:
Fatal error compiling: error: bad value for --module-version option: 'JAPICMP-OLD'

Therefore this pull request changes the dummy version number format, so that it is not rejected anymore.

The workflow is currently failing for the jpms-test module:
> Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.13.0:compile (default-compile) on project jpms-test:
> Fatal error compiling: error: bad value for --module-version option: 'JAPICMP-OLD'

Therefore this commit changes the dummy version number format.
@eamonnmcmanus
Copy link
Member

Thanks!

It seems like the API compatibility workflow is not failing with this PR. Not sure if you want to update the description.

@Marcono1234
Copy link
Collaborator Author

Oh you are right; I forgot to consider that since this changes the workflow, it applies directly (and not delayed only on the next pull request).

@eamonnmcmanus eamonnmcmanus merged commit 3b296cc into google:main Jun 1, 2024
11 checks passed
@Marcono1234 Marcono1234 deleted the marcono1234/api-compatibility-workflow-fix branch June 1, 2024 00:38
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