add jdk toolchain ITs by hboutemy · Pull Request #26 · apache/maven-toolchains-plugin (original) (raw)

Skip to content

Sign in

Appearance settings

View all features

View all solutions

View all resources

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Sign in

Sign up

Appearance settings

apache / maven-toolchains-plugin Public

Additional navigation options

Merged

hboutemy

merged 1 commit into

masterfrom

it-jdk

Mar 4, 2024

ConversationCommits (1)ChecksFiles changed

Merged

add jdk toolchain ITs#26

hboutemy

merged 1 commit into

masterfrom

it-jdk

Conversation

@hboutemy

@hboutemy hboutemy commented

Mar 3, 2024

Copy link Copy Markdown

Member

and improve documentation based on output found when running the new ITs that show many different cases

@hboutemy

[add jdk toolchain ITs](/apache/maven-toolchains-plugin/pull/26/commits/bf6fcd40742e5cb974c00e1af6bbd14db1efc308 "add jdk toolchain ITs")

[bf6fcd4](/apache/maven-toolchains-plugin/pull/26/commits/bf6fcd40742e5cb974c00e1af6bbd14db1efc308)

@hboutemy hboutemy requested a review from elharo

March 3, 2024 17:15

elharo

elharo approved these changes Mar 3, 2024

View reviewed changes

@elharo elharo left a comment

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.

Nice!

@hboutemy hboutemy merged commit 91cda5f into master

Mar 4, 2024

@hboutemy hboutemy deleted the it-jdk branch

March 4, 2024 06:17

@olamy olamy added the maintenance label

Apr 11, 2024

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

Sign up for free to join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@elharo elharo elharo approved these changes

Assignees

No one assigned

Labels

maintenance

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hboutemy @elharo @olamy