Skip to content

[java] Support groovy installations - #779

Merged
samruddhikhandale merged 5 commits into
devcontainers:mainfrom
Ash258:java/groovy_test_fixes
Jan 22, 2024
Merged

[java] Support groovy installations#779
samruddhikhandale merged 5 commits into
devcontainers:mainfrom
Ash258:java/groovy_test_fixes

Conversation

@Ash258

@Ash258 Ash258 commented Dec 9, 2023

Copy link
Copy Markdown
Contributor

Tests are failing for some reason with version 19. I used 21 instead to make tests work again

image

Fix java test failing with non existent 19 version of `open` distro
@Ash258
Ash258 requested a review from a team as a code owner December 9, 2023 11:38
@Ash258

Ash258 commented Dec 9, 2023

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@samruddhikhandale samruddhikhandale left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great, thank you for adding the super helpful Feature option.

Left minor comments, besides that looks great!

Comment thread src/java/devcontainer-feature.json Outdated
Comment thread src/java/devcontainer-feature.json Outdated
Comment thread test/java/scenarios.json

@samruddhikhandale samruddhikhandale left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, thank you! 🚀

@samruddhikhandale
samruddhikhandale merged commit 76ffd41 into devcontainers:main Jan 22, 2024
@Ash258
Ash258 deleted the java/groovy_test_fixes branch January 23, 2024 00:45
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