Skip to content

Address Linting warnings #254

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.

Sign up for GitHub

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

Jump to bottom
Merged
Kilo59 merged 3 commits into from
Apr 2, 2020
Merged

Address Linting warnings #254

Kilo59 merged 3 commits into from
Apr 2, 2020

Conversation

@Kilo59
Copy link
Collaborator

@Kilo59 Kilo59 commented Apr 2, 2020

  • address outstanding linting warning with fixes or error suppression
  • re-enable lint CI stage

Kilo59 added 3 commits April 1, 2020 23:13
@Kilo59
lint fixes
a33ce8f
@Kilo59
re-enable CI failure on lint warnings
94fe650
@Kilo59
edit lint config
2dc8d3d 
* white list pydantic
* allow fstring logging
* allow TODO
* temporarily allow duplicate code
@Kilo59 Kilo59 added the dev Issue related to ease of development and code smells label Apr 2, 2020
@Kilo59 Kilo59 changed the title Address Linting warning Address Linting warnings Apr 2, 2020
@Kilo59 Kilo59 merged commit 2f95275 into ExpDev07:master Apr 2, 2020
@Kilo59 Kilo59 deleted the linting branch April 2, 2020 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

dev Issue related to ease of development and code smells

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Kilo59