Skip to content

BUILD-8568: Fix file path interpretation in windows#42

Merged
jayadeep-km-sonarsource merged 1 commit intomasterfrom
feat/jd/BUILD-8568-windows-build-fix
Aug 6, 2025
Merged

BUILD-8568: Fix file path interpretation in windows#42
jayadeep-km-sonarsource merged 1 commit intomasterfrom
feat/jd/BUILD-8568-windows-build-fix

Conversation

@jayadeep-km-sonarsource
Copy link
Copy Markdown
Contributor

@jayadeep-km-sonarsource jayadeep-km-sonarsource commented Aug 6, 2025

BUILD-8568

BUILD-8568: Fix file path interpretation in windows

The API path was getting intrepreted as file path in windows and causing issues with windows builds

Signed-off-by: Jayadeep Kinavoor Madam <jayadeep.kinavoormadam@sonarsource.com>
@jayadeep-km-sonarsource jayadeep-km-sonarsource force-pushed the feat/jd/BUILD-8568-windows-build-fix branch from 5432371 to 3ed1642 Compare August 6, 2025 07:46
@jayadeep-km-sonarsource jayadeep-km-sonarsource marked this pull request as ready for review August 6, 2025 07:47
@jayadeep-km-sonarsource jayadeep-km-sonarsource requested a review from a team August 6, 2025 07:47
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Aug 6, 2025

🤖 Pull Request summary

Fixes GitHub API URL path format by removing leading slash.

• Removed leading slash from PROPERTIES_API_URL to correct the GitHub API endpoint format
• Changed from /repos/... to repos/... for proper API path construction

Review focus: Verify this resolves any API call failures and test the build number retrieval functionality.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@jayadeep-km-sonarsource jayadeep-km-sonarsource merged commit f407ec7 into master Aug 6, 2025
10 checks passed
@jayadeep-km-sonarsource jayadeep-km-sonarsource deleted the feat/jd/BUILD-8568-windows-build-fix branch August 6, 2025 07:50
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