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

Allow for file paths on the input argument (not only directories) #208

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

nxtstep
Copy link

@nxtstep nxtstep commented Oct 31, 2023

The detekt-cli allows for a comma-separated list of paths - including single files - to run the checks on.

This PR makes sure we can also use this feature with this plugin.

Following up on: #203 where multiple directories were supported, however, the detekt-cli not only accepts directories, but single file(s) too. 🚀

The detekt-cli allows for a comma-separated list of paths - including single files - to run the checks on
@nxtstep
Copy link
Author

nxtstep commented Nov 1, 2023

@Ozsie would you mind having a look at this PR and possibly approve it 😇 🙏

@Ozsie Ozsie merged commit adc8b79 into Ozsie:main Nov 1, 2023
13 of 14 checks passed
@nxtstep nxtstep deleted the fix-input-paths branch November 1, 2023 12:02
@nxtstep
Copy link
Author

nxtstep commented Nov 2, 2023

@Ozsie thanks a lot for merging 🙏 are you planning a next release soon[ish]? 😇

@Ozsie
Copy link
Owner

Ozsie commented Nov 2, 2023

@Ozsie thanks a lot for merging 🙏 are you planning a next release soon[ish]? 😇

Yes, I was hoping to release as soon as I merged the latest detekt version, but I got sidetracked by work and life.

I'll push the button in a few minutes.

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