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

Fix GitHub Actions Errors #427

Merged
merged 3 commits into from
Jun 16, 2023
Merged

Fix GitHub Actions Errors #427

merged 3 commits into from
Jun 16, 2023

Conversation

VKTB
Copy link
Contributor

@VKTB VKTB commented Jun 2, 2023

Description

Fixes all the GitHub Action-related errors (https://github.com/ral-facilities/datagateway-api/actions/runs/5146527468). Also ignores the werkzeug vulnerability as the fix for this requires Python 3.7 or above.

Testing Instructions

  • Review code
  • Check GitHub Actions build
  • If icatdb Generator Script Consistency Test CI job fails, is this because of a deliberate change made to the script to change generated data (which isn't actually a problem) or is here an underlying issue with the changes made?
  • Review changes to test coverage
  • Does this change mean a new patch, minor or major version should be made? If so, does one of the commit messages feature fix:, feat: or BREAKING CHANGE: so a release is automatically made via GitHub Actions upon merge?

@codecov
Copy link

codecov bot commented Jun 2, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (c1c49b9) 96.80% compared to head (623a4b3) 96.80%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #427   +/-   ##
=======================================
  Coverage   96.80%   96.80%           
=======================================
  Files          40       40           
  Lines        3375     3375           
  Branches      317      317           
=======================================
  Hits         3267     3267           
  Misses         80       80           
  Partials       28       28           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Collaborator

@MRichards99 MRichards99 left a comment

Choose a reason for hiding this comment

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

I'm happy with the change but the test coverage seems to be -2% and a semantic commit needs to be added. Could you investigate the test coverage and add a sematic commit please?

@VKTB VKTB force-pushed the fix-gh-action-errors branch from ad8fa9c to be37a71 Compare June 12, 2023 12:40
@VKTB
Copy link
Contributor Author

VKTB commented Jun 12, 2023

I'm happy with the change but the test coverage seems to be -2% and a semantic commit needs to be added. Could you investigate the test coverage and add a sematic commit please?

I honestly have no idea why the coverage decreased by 2%. I spent some time looking at this but could not work it out.

@VKTB
Copy link
Contributor Author

VKTB commented Jun 12, 2023

@MRichards99 Looks okay now. It updated after the CI ran again.

@VKTB VKTB requested a review from MRichards99 June 16, 2023 07:29
@VKTB VKTB merged commit ea2db01 into main Jun 16, 2023
@VKTB VKTB deleted the fix-gh-action-errors branch June 16, 2023 07:34
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.

2 participants