Skip to content
This repository was archived by the owner on Sep 8, 2025. It is now read-only.

Add lgtm, do other stuff#19

Merged
jepler merged 7 commits into
mainfrom
lgtm
Nov 4, 2021
Merged

Add lgtm, do other stuff#19
jepler merged 7 commits into
mainfrom
lgtm

Conversation

@jepler
Copy link
Copy Markdown
Owner

@jepler jepler commented Nov 3, 2021

No description provided.

@jepler jepler self-assigned this Nov 3, 2021
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 3, 2021

Codecov Report

Merging #19 (bd7bffc) into main (f55d1a8) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##              main       #19   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines         1040      1041    +1     
  Branches       205       205           
=========================================
+ Hits          1040      1041    +1     
Impacted Files Coverage Δ
wwvb/decode.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f55d1a8...bd7bffc. Read the comment docs.

@lgtm-com
Copy link
Copy Markdown

lgtm-com Bot commented Nov 3, 2021

This pull request introduces 1 alert and fixes 1 when merging bd7bffc into f55d1a8 - view on LGTM.com

new alerts:

  • 1 for Non-callable called

fixed alerts:

  • 1 for Unreachable code

@jepler jepler merged commit b445bc9 into main Nov 4, 2021
@jepler jepler deleted the lgtm branch November 30, 2021 01:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants