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

Humidex #33

Merged
Merged

Conversation

SusannePfeifer007
Copy link
Contributor

Bitte nimm den auf

@ludwiglierhammer
Copy link
Collaborator

Sobald du die Tests angepasst und die Mergekonflikte behoben hast, kann ich den PR mergen.

@codecov-commenter
Copy link

codecov-commenter commented Oct 23, 2023

Codecov Report

Merging #33 (7c64e9a) into master (c82a674) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
+ Coverage   94.77%   94.79%   +0.02%     
==========================================
  Files          14       14              
  Lines        1262     1268       +6     
==========================================
+ Hits         1196     1202       +6     
  Misses         66       66              
Flag Coverage Δ
unittests 94.79% <100.00%> (+0.02%) ⬆️

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

Files Coverage Δ
index_calculator/_indices.py 100.00% <100.00%> (ø)

Copy link
Collaborator

Choose a reason for hiding this comment

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

i added setup.cfg for convenience

Comment on lines +4 to +12
E203
# module level import not at top of file
E402
# line too long - let black worry about that
E501
# do not assign a lambda expression, use a def
E731
# line break before binary operator
W503
Copy link
Collaborator

Choose a reason for hiding this comment

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

we will ignore these conventions

@larsbuntemeyer
Copy link
Collaborator

@SusannePfeifer007 check mal pre-commit! Das ist mega nützlich und man muss sich keine Gedanken mehr um Formatierung machen!

@larsbuntemeyer larsbuntemeyer merged commit bce0df7 into climate-service-center:master Oct 23, 2023
3 checks passed
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.

4 participants