Skip to content

Do not use relative path inside CLI script#1898

Merged
eric-maynard merged 2 commits intoapache:mainfrom
eric-maynard:issue-1868
Jun 14, 2025
Merged

Do not use relative path inside CLI script#1898
eric-maynard merged 2 commits intoapache:mainfrom
eric-maynard:issue-1868

Conversation

@eric-maynard
Copy link
Contributor

@eric-maynard eric-maynard commented Jun 13, 2025

Issue #1868 reported that the Polaris script can fail when it's run from an unexpected path. The recent addition of a reference to ./gradlew looks incorrect here, and should be changed to use an absolute path.

Fixes #1868

dimas-b
dimas-b previously approved these changes Jun 13, 2025
@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Jun 13, 2025
HonahX
HonahX previously approved these changes Jun 13, 2025
singhpk234
singhpk234 previously approved these changes Jun 13, 2025
@eric-maynard eric-maynard dismissed stale reviews from HonahX and dimas-b via bdba865 June 13, 2025 20:11
@eric-maynard eric-maynard merged commit 2d3d0d0 into apache:main Jun 14, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Jun 14, 2025
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.

Local regression test (test_cli.py) failed

5 participants