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

Migrate from 'Travis CI' to 'GitHub Actions' (...) #36

Merged
merged 1 commit into from
Nov 8, 2020
Merged

Conversation

urda
Copy link
Owner

@urda urda commented Nov 8, 2020

Changes

  • Move codecov config into .github.
  • Convert fromtravis to GitHub Actions runners.
  • Update gitignore for package building results ignoring.
  • Cleanup Makefile:
    • ... We will return docs some day as a make command ...
  • Update README.
  • Unpin pylint:
    • ... update any pylint error reports.
  • Update setup.py for further cleanup
  • Update touched copyright dates.
  • Fix up any type hint / code smell issues during PR.
  • Upgrade version checker via rip from this commit

Notes

Why not have python3.9 ?

PyLint was not happy:

make pylint
pylint --rcfile=./.pylintrc --reports=y --output-format=text nistbeacon
************* Module nistbeacon.nistbeaconvalue
nistbeacon/nistbeaconvalue.py:373:41: E1136: Value 'Optional' is unsubscriptable (unsubscriptable-object)
************* Module nistbeacon.nistbeacon
nistbeacon/nistbeacon.py:61:43: E1136: Value 'Optional' is unsubscriptable (unsubscriptable-object)

See also:

@codecov-io
Copy link

codecov-io commented Nov 8, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@43e0c3d). Click here to learn what that means.
The diff coverage is 91.66%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #36   +/-   ##
=========================================
  Coverage          ?   99.47%           
=========================================
  Files             ?        4           
  Lines             ?      191           
  Branches          ?        0           
=========================================
  Hits              ?      190           
  Misses            ?        1           
  Partials          ?        0           
Impacted Files Coverage Δ
nistbeacon/nistbeacon.py 97.95% <83.33%> (ø)
nistbeacon/nistbeaconcrypto.py 100.00% <100.00%> (ø)
nistbeacon/nistbeaconvalue.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 43e0c3d...681f9cd. Read the comment docs.

@urda urda force-pushed the develop branch 6 times, most recently from 860a04e to b8d9863 Compare November 8, 2020 03:32
@urda
Copy link
Owner Author

urda commented Nov 8, 2020

Made corrections for pylint upgrade:

 pylint --rcfile=./.pylintrc --reports=y --output-format=text nistbeacon
************* Module nistbeacon.nistbeacon
nistbeacon/nistbeacon.py:23:0: R0205: Class 'NistBeacon' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
nistbeacon/nistbeacon.py:69:12: R1705: Unnecessary "else" after "return" (no-else-return)
nistbeacon/nistbeacon.py:80:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
nistbeacon/nistbeacon.py:160:8: R1705: Unnecessary "else" after "return" (no-else-return)
************* Module nistbeacon.nistbeaconvalue
nistbeacon/nistbeaconvalue.py:26:0: R0205: Class 'NistBeaconValue' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
************* Module nistbeacon.nistbeaconcrypto
nistbeacon/nistbeaconcrypto.py:26:0: R0205: Class 'NistBeaconCrypto' inherits from object, can be safely removed from bases in python3 (useless-object-inheritance)
nistbeacon/nistbeaconcrypto.py:186:21: R1719: The if expression can be replaced with 'test' (simplifiable-if-expression)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants