Skip to content

Conversation

@ab93
Copy link
Member

@ab93 ab93 commented Sep 21, 2022

  • Added linting
  • Format code to pass linting
  • Added flake8 dependency
  • Added codeowners and other templates

Signed-off-by: Avik Basu avikbasu93@gmail.com

@codecov
Copy link

codecov bot commented Sep 21, 2022

Codecov Report

Merging #9 (3a8b07c) into main (fd88178) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   87.54%   87.54%           
=======================================
  Files           9        9           
  Lines         273      273           
  Branches       21       21           
=======================================
  Hits          239      239           
  Misses         32       32           
  Partials        2        2           
Impacted Files Coverage Δ
pynumaflow/function/_dtypes.py 100.00% <100.00%> (ø)
pynumaflow/sink/_dtypes.py 100.00% <100.00%> (ø)

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

@ab93 ab93 linked an issue Sep 21, 2022 that may be closed by this pull request
@ab93 ab93 self-assigned this Sep 21, 2022
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
@ab93 ab93 marked this pull request as ready for review September 21, 2022 23:05
@ab93 ab93 requested review from jy4096 and vigith September 21, 2022 23:05
@ab93 ab93 merged commit 0e1844f into main Sep 22, 2022
@ab93 ab93 deleted the lint branch September 22, 2022 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set up linting in workflow

2 participants