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

feat: allow ignoring commits from certain users #3666

Open
4 tasks
jgresty opened this issue Sep 18, 2023 · 2 comments
Open
4 tasks

feat: allow ignoring commits from certain users #3666

jgresty opened this issue Sep 18, 2023 · 2 comments
Labels

Comments

@jgresty
Copy link

jgresty commented Sep 18, 2023

Expected Behavior

Some commits are raised by bots or other scripts that may not necessarily pass strict rulesets. It should be possible to bypass linting commits by some specified users so that commits made by some automated process do not need to be manually fixed up.

Current Behavior

It is only possible to ignore commits by the content of the messages.

Affected packages

  • cli
  • core
  • prompt
  • config-angular

Possible Solution

Extend the ignores config matcher to include additional metadata. For this use case only user is required but there may be other use cases:

ignores: [
  (message, user) => user.includes('[bot]@users.noreply.github.com')
]

Context

We use automated processes to upgrade pull requests, the commits raised in this way contain a URL that sometimes is longer than the maximum line length. Since these are automated we don't particularly care about perfect formatting so want to ignore any commits raised by bot users.

@Clockwork-Muse

This comment was marked as off-topic.

@sdavids

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

3 participants