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

codacity: upgrade to latest/main github action version #585

Merged
merged 1 commit into from
Jan 31, 2022

Conversation

juarezr
Copy link
Member

@juarezr juarezr commented Jan 31, 2022

This PR has the objective of upgrading Codacy Analysis CLI GitHub Action

Changes

  1. Upgraded codacy/codacy-analysis-cli-action from version v1.1.0 to master described in marketplace page
  2. Upgraded github/codeql-action/upload-sarif from version v2 to master as described in marketplace page
  3. Improved the comments in the file documenting the scenario usage
  4. First step to fix the issues with this action in the Github Actions and PRs

Checklist

Checklist for for pull requests including new code and/or changes to existing code...

  • Code
    • Includes unit tests
    • New functions have docstrings with examples that can be run with doctest
    • New functions are included in API docs
    • Docstrings include notes for any changes to API or behaviour
    • All changes documented in docs/changes.rst
  • Testing
    • (Optional) Tested local against remote servers
    • Github CI passes (unit tests run under Linux/Windows/MacOS)
    • AppVeyor CI passes (unit tests run under Windows)
    • Unit test coverage has not decreased (see Coveralls)
  • Changes
    • (Optional) Just a proof of concept
    • (Optional) Work in progress
    • Ready to review
    • Ready to merge

@juarezr juarezr added dependencies Pull requests that update a dependency file Build CI CD Issues affecting github pypi readthedocs coveralls conda-forge labels Jan 31, 2022
@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.871% when pulling 54ce14a on juarezr:codacity_upgrade into 9f12eaf on petl-developers:master.

@juarezr juarezr merged commit 860a333 into petl-developers:master Jan 31, 2022
@juarezr juarezr deleted the codacity_upgrade branch January 31, 2022 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build CI CD Issues affecting github pypi readthedocs coveralls conda-forge dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants