Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Testing #13

Merged
merged 21 commits into from
Dec 20, 2022
Merged

Testing #13

merged 21 commits into from
Dec 20, 2022

Conversation

mjohnson541
Copy link
Contributor

Add a system for automatically running software tests to ensure things don't break.

@lgtm-com
Copy link

lgtm-com bot commented Dec 8, 2022

This pull request introduces 8 alerts and fixes 22 when merging 5421cf8 into 66cacae - view on LGTM.com

new alerts:

  • 5 for Unused import
  • 1 for Except block handles 'BaseException'
  • 1 for Modification of parameter with default
  • 1 for 'import *' may pollute namespace

fixed alerts:

  • 11 for Unused import
  • 10 for Module-level cyclic import
  • 1 for Conflicting attributes in base classes

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@lgtm-com
Copy link

lgtm-com bot commented Dec 8, 2022

This pull request introduces 8 alerts and fixes 22 when merging 0b18d7a into 66cacae - view on LGTM.com

new alerts:

  • 5 for Unused import
  • 1 for Except block handles 'BaseException'
  • 1 for Modification of parameter with default
  • 1 for 'import *' may pollute namespace

fixed alerts:

  • 11 for Unused import
  • 10 for Module-level cyclic import
  • 1 for Conflicting attributes in base classes

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

@mjohnson541 mjohnson541 force-pushed the testing branch 3 times, most recently from 7838223 to 6cc408a Compare December 9, 2022 22:33
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@40bfa0a). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #13   +/-   ##
=========================================
  Coverage          ?   60.59%           
=========================================
  Files             ?       11           
  Lines             ?     2449           
  Branches          ?        0           
=========================================
  Hits              ?     1484           
  Misses            ?      965           
  Partials          ?        0           

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

@mjohnson541 mjohnson541 deleted the testing branch December 20, 2022 20:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants