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

Update YamLint #173

Merged
merged 2 commits into from
Apr 18, 2022
Merged

Update YamLint #173

merged 2 commits into from
Apr 18, 2022

Conversation

Potherca
Copy link
Member

@Potherca Potherca commented Apr 18, 2022

  1. Currently, YamlLint complains about the line-length in securitycheck.yml GitHub Worflow.

    image

    This MR resolves that violation by adding a comment telling YamlLint to shut the fuck up kindly ignore that specific line.

    (see https://yamllint.readthedocs.io/en/stable/disable_with_comments.html for documentation details).

  2. As using on: as a key name is no longer marked as a violation (by the truthy rule configuration in the .yamllint file), the following diable comment has been removed in all files that had it:

    # yamllint disable-line rule:truthy

@jrfnl jrfnl merged commit a1b6e4e into master Apr 18, 2022
@jrfnl jrfnl deleted the fix/yamllintlinelength branch April 18, 2022 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants