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 linter config to match flintlock #115

Merged
merged 5 commits into from
Jan 31, 2022
Merged

Update linter config to match flintlock #115

merged 5 commits into from
Jan 31, 2022

Conversation

yitsushi
Copy link
Contributor

What this PR does / why we need it:

As the repo is growing, it will be harder and harder to do this. Right now we are focused on the whole capmvm+flintlock workflow and we don't have heavy changes, that's good to avoid a lot of conflicts.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
fixes #31

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@yitsushi yitsushi added the kind/refactor Only refactoring changes label Jan 25, 2022
@yitsushi yitsushi changed the title 31 update linter Update linter config to match flintlock Jan 25, 2022
@yitsushi yitsushi requested a review from richardcase January 28, 2022 11:34
@yitsushi yitsushi merged commit 30b07dd into main Jan 31, 2022
@yitsushi yitsushi deleted the 31-update-linter branch January 31, 2022 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/refactor Only refactoring changes size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update linter config to match flintlock
2 participants