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

fix: invalid kotlin build report config directions #4472

Merged

Conversation

EddieRingle
Copy link
Contributor

Wrapping the json directory property with quotes results in an error:

Kotlin build report cannot be created: '"my/directory/path"' is a file or do not have permissions to create

Hit this error while following these docs and also found a relevant ticket on YouTrack that was already closed: https://youtrack.jetbrains.com/issue/KT-71049/Cannot-get-build-report-is-a-file-or-do-not-have-permissions-to-create

@EddieRingle EddieRingle marked this pull request as ready for review October 3, 2024 01:03
@EddieRingle EddieRingle requested a review from a team as a code owner October 3, 2024 01:03
@sarahhaggarty sarahhaggarty self-assigned this Oct 3, 2024
@nav-nav
Copy link

nav-nav commented Oct 7, 2024

Looks good, thank you.

@sarahhaggarty sarahhaggarty requested a review from nav-nav October 7, 2024 11:27
Copy link

@nav-nav nav-nav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

Wrapping the json directory property with quotes results in an error.

RE: KT-71049

Signed-off-by: Eddie Ringle <[email protected]>
@sarahhaggarty sarahhaggarty force-pushed the er/build-report-config-fix branch from 79ffba7 to 3b7207c Compare October 8, 2024 10:37
@sarahhaggarty sarahhaggarty merged commit d2fb6e0 into JetBrains:master Oct 8, 2024
4 checks passed
@sarahhaggarty
Copy link
Collaborator

Thanks for your contribution @EddieRingle ! I updated your commit to include the fix on another page which has the same sample.

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.

3 participants