Skip to content
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.

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

Path on windows platform #440

Merged
merged 3 commits into from
Jul 25, 2024
Merged

Path on windows platform #440

merged 3 commits into from
Jul 25, 2024

Conversation

gyurib
Copy link
Contributor

@gyurib gyurib commented Jul 24, 2024

These changes fix the issue reported in #394
The solution was to change the paths to be system independent.
Tested in windows and unix platforms

gyurib added 2 commits July 24, 2024 16:28
URL double encoding in Chrome

These changes fix the issue reported in zalando#394 
The solution was to change the paths to be system independent.
Tested in windows and unix platforms
@gyurib gyurib requested a review from a team as a code owner July 24, 2024 13:39
Copy link

invalid team ID

The team ID in your .zappr.yaml file does not appear to be valid. Please, fix
this before team ID checks will be added back into ComPR's specification check.

You can follow this guideline for help.

1 similar comment
Copy link

invalid team ID

The team ID in your .zappr.yaml file does not appear to be valid. Please, fix
this before team ID checks will be added back into ComPR's specification check.

You can follow this guideline for help.

@gyurib gyurib closed this Jul 24, 2024
@gyurib gyurib reopened this Jul 24, 2024
asaarilahti
asaarilahti previously approved these changes Jul 24, 2024
@asaarilahti
Copy link
Contributor

asaarilahti commented Jul 24, 2024

Thanks for your contribution Gyuri! Would you be able to run ./gradlew :spotlessApply to make the formatting to pass the checks?

@gyurib
Copy link
Contributor Author

gyurib commented Jul 24, 2024 via email

@asaarilahti asaarilahti merged commit d57e60a into zalando:master Jul 25, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants