Skip to content

fix: install maven in windows tests#8639

Merged
seshubaws merged 2 commits intoaws:developfrom
seshubaws:maven-tests
Feb 9, 2026
Merged

fix: install maven in windows tests#8639
seshubaws merged 2 commits intoaws:developfrom
seshubaws:maven-tests

Conversation

@seshubaws
Copy link
Copy Markdown
Contributor

Which issue(s) does this change fix?

Error in windows integ tests

mvn --version
'mvn' is not recognized as an internal or external command,
operable program or batch file.
Command exited with code 1

Why is this change necessary?

How does it address the issue?

What side effects does this change have?

Mandatory Checklist

PRs will only be reviewed after checklist is complete

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@seshubaws seshubaws requested a review from a team as a code owner February 9, 2026 21:56
@bnusunny
Copy link
Copy Markdown
Contributor

bnusunny commented Feb 9, 2026

Nit: do we want to specify a version for maven?

@seshubaws seshubaws enabled auto-merge February 9, 2026 22:35
@seshubaws seshubaws added this pull request to the merge queue Feb 9, 2026
Merged via the queue into aws:develop with commit b9aa1ab Feb 9, 2026
44 checks passed
@seshubaws seshubaws deleted the maven-tests branch February 10, 2026 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants