Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add rki include/exclude flag #319

Merged
merged 39 commits into from
Dec 15, 2021
Merged

feat: add rki include/exclude flag #319

merged 39 commits into from
Dec 15, 2021

Conversation

AKBrueggemann
Copy link
Collaborator

Expected commit message structure

Commit messages should be structured as follows:

<type>[optional scope]: <description>

[optional body]

[optional footer]

Type

Must be one of the following:

  • build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
  • ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
  • docs: Documentation only changes
  • feat: A new feature
  • fix: A bug fix
  • perf: A code change that improves performance
  • refactor: A code change that neither fixes a bug nor adds a feature
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • test: Adding missing tests or correcting existing tests

Examples

Commit message with description and breaking change in body

feat: allow provided config object to extend other configs

BREAKING CHANGE: `extends` key in config file is now used for extending other config files

Commit message with no body

docs: correct spelling of CHANGELOG

Commit message with scope

feat(lang): added polish language

Commit message for a fix using an (optional) issue number.

fix: minor typos in code

see the issue for details on the typos fixed

fixes issue #12

@AKBrueggemann AKBrueggemann linked an issue Nov 8, 2021 that may be closed by this pull request
@AKBrueggemann AKBrueggemann changed the title feat: rki flag feat: add rki include/exclude flag Nov 8, 2021
Copy link
Contributor

@thomasbtf thomasbtf left a comment

Choose a reason for hiding this comment

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

Again, good Stuff. If we address the requests below, we are good to go 👍

@thomasbtf thomasbtf merged commit d2e6d20 into master Dec 15, 2021
@thomasbtf thomasbtf deleted the rki-flag branch December 15, 2021 11:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add RKI exclusion flag
2 participants