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 further linter files #13

Merged
merged 9 commits into from
Feb 22, 2023
Merged

Conversation

brainelectronics
Copy link
Owner

Added

  • .editorconfig for common editor settings, see Add other linter files #12
  • .yamllint to lint all used YAML files, see Add other linter files #12
  • codecov.yaml to specify further settings and limits for the coverage
  • yamllint package to the requirements-test.txt file
  • Run YAML linter on test workflow

Changed

  • Fixed uncovered YAML syntax issues in all workflow files
  • Removed unused files from .gitignore file

@brainelectronics brainelectronics added the enhancement New feature or request label Feb 22, 2023
@brainelectronics brainelectronics self-assigned this Feb 22, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #13 (56a9975) into main (8f3cb91) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff            @@
##              main       #13   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines           14        14           
  Branches         1         1           
=========================================
  Hits            14        14           
Flag Coverage Δ
unittests 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Owner Author

@brainelectronics brainelectronics left a comment

Choose a reason for hiding this comment

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

LGTMyself

.github/workflows/release.yml Show resolved Hide resolved
@brainelectronics brainelectronics merged commit 7799cdd into main Feb 22, 2023
@brainelectronics brainelectronics deleted the feature/add-further-linter-files branch February 22, 2023 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants