Skip to content

Remove unnecessary path modification #13671

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.

Sign up for GitHub

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

Jump to bottom
Open
yeikel wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Remove unnecessary path modification #13671

yeikel wants to merge 1 commit into from
+0 −1

Conversation

@yeikel
Copy link
Contributor

@yeikel yeikel commented Nov 30, 2025

What are you trying to accomplish?

This manual path change is not needed because Java is installed globally and is already in the path

Checklist

  • I have run the complete test suite to ensure all tests and linters pass.
  • I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
  • I have written clear and descriptive commit messages.
  • I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
  • I have ensured that the code is well-documented and easy to understand.

@yeikel yeikel requested a review from a team as a code owner November 30, 2025 20:00
@github-actions github-actions bot added the L: java:maven Maven packages via Maven label Nov 30, 2025
@yeikel yeikel changed the title Remove unnecessary step Remove unnecessary path modification Nov 30, 2025
@yeikel yeikel force-pushed the remove-path-add branch 3 times, most recently from 2d28ae4 to 48dc516 Compare December 4, 2025 01:18
@markhallen
Copy link
Contributor

markhallen commented Dec 4, 2025

@yeikel could you resolve the merge conflict please

markhallen
markhallen previously approved these changes Dec 4, 2025
View reviewed changes
@yeikel
Copy link
Contributor Author

yeikel commented Dec 4, 2025

@markhallen Please check now

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

Reviewers

@markhallen markhallen markhallen left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

L: java:maven Maven packages via Maven

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yeikel @markhallen