Skip to content

Update maven test output#225

Closed
amazimbe wants to merge 1 commit intomainfrom
amazimbe/update-maven-test
Closed

Update maven test output#225
amazimbe wants to merge 1 commit intomainfrom
amazimbe/update-maven-test

Conversation

@amazimbe
Copy link
Copy Markdown
Contributor

@amazimbe amazimbe commented Sep 10, 2024

Link to issue: #226

The updated test started failing after this change: dependabot/dependabot-core#10558 and this PR updates the contents of the output section so that the tests pass again.

@amazimbe amazimbe requested a review from a team as a code owner September 10, 2024 13:55
@amazimbe amazimbe force-pushed the amazimbe/update-maven-test branch from f38bf94 to a160154 Compare September 10, 2024 14:11
@amazimbe amazimbe marked this pull request as draft September 10, 2024 14:18
@amazimbe amazimbe changed the title Update maven test security advisory versions Update maven test output Sep 10, 2024
@amazimbe amazimbe requested a review from kbukum1 September 10, 2024 15:19
@amazimbe amazimbe force-pushed the amazimbe/update-maven-test branch from a160154 to 7f3620d Compare September 11, 2024 14:34
@amazimbe amazimbe marked this pull request as ready for review September 11, 2024 15:07
metadata:
packaging_type: pom
requirement: 1.5.12.RELEASE
requirement: 1.5.11.RELEASE
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not something big since the pattern same, just wanted to check if there is a reason we didn't use the same version 1.5.12.RELEASE

workflow_dispatch:
pull_request:
branches: ["main"]
branches: ["amazimbe/use-new-maven-version"]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please let's not forget to switch back that before the merging.

@amazimbe
Copy link
Copy Markdown
Contributor Author

Closing in favour of #228

@amazimbe amazimbe closed this Sep 18, 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

Successfully merging this pull request may close these issues.

2 participants