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 exclude option in selene.toml #425

Merged
merged 5 commits into from
Sep 25, 2022
Merged

Conversation

captalbator
Copy link
Contributor

This implements an exclude glob option in selene.toml which excludes any files matching with the globs from being linted.

Copy link
Owner

@Kampfkarren Kampfkarren left a comment

Choose a reason for hiding this comment

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

Add it to CHANGELOG.md and to the documentation for selene.toml as well

selene/Cargo.toml Show resolved Hide resolved
selene/src/main.rs Outdated Show resolved Hide resolved
selene/src/main.rs Outdated Show resolved Hide resolved
selene/src/main.rs Show resolved Hide resolved
@Kampfkarren
Copy link
Owner

Kampfkarren commented Sep 19, 2022

Conflicts with main now

@Kampfkarren Kampfkarren merged commit 5d47888 into Kampfkarren:main Sep 25, 2022
@OverHash
Copy link
Contributor

OverHash commented Oct 3, 2022

Should the error! calls here use std::process:exit(1) rather than return after errorring? That would be consistent with the other logic in src/main.rs..

@Kampfkarren
Copy link
Owner

@OverHash Yeah, didn't catch it

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