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

Add relative option similar to CLI's --relative #573

Conversation

pschichtel
Copy link
Contributor

We use this gradle plugin in GitLab-CI and currently it always produces absolute paths which leads to broken links in the code quality report.

This PR adds the option relative analog to ktlint's CLI's --relative option, which does the same thing: Instead of outputting absolute paths in the report, the reports are made relative to the root project (ktlint make it relative to the workdir).

Currently this is only applied to the actual report file output, not the console output to still have clickable file paths in IntelliJ.

@pschichtel
Copy link
Contributor Author

I tested this in a large multi-module JVM-only project.

@pschichtel
Copy link
Contributor Author

@JLLeitschuh just a friendly ping to make sure this PR is not accidentally overseen (I tend to miss/not receive notifications).

@pschichtel pschichtel force-pushed the feature/relative-paths-in-reports branch from bd89705 to 643f38b Compare March 16, 2022 10:57
@JLLeitschuh
Copy link
Owner

I actually did miss this, could you rebase this PR for me?

@pschichtel pschichtel force-pushed the feature/relative-paths-in-reports branch from 643f38b to a18bc8d Compare April 21, 2022 11:45
@pschichtel
Copy link
Contributor Author

@JLLeitschuh done

@pschichtel
Copy link
Contributor Author

@JLLeitschuh I see the windows check failed, but it doesn't look like that is related to this change. can you confirm?

@JLLeitschuh JLLeitschuh merged commit 505c312 into JLLeitschuh:master May 3, 2022
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