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

Update dependencies to prevent Poetry warning #231

Merged
merged 1 commit into from
May 5, 2021

Conversation

MRichards99
Copy link
Collaborator

@MRichards99 MRichards99 commented May 5, 2021

This PR will close #230

Description

This is a very small PR to update the dependencies in the API, done via poetry update. This will prevent the following warning:

Warning: The lock file is not up to date with the latest changes in pyproject.toml. You may be getting outdated dependencies. Run update to update them.

I believe this was caused when dealing with a merge conflict in poetry.lock.

Testing Instructions

Just make sure you can do a poetry install on this branch without any issues

  • Review code
  • Check GitHub Actions build
  • Review changes to test coverage

Agile Board Tracking

Connect to #230

@codecov
Copy link

codecov bot commented May 5, 2021

Codecov Report

Merging #231 (3ef5f3a) into master (0c68549) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #231   +/-   ##
=======================================
  Coverage   89.45%   89.45%           
=======================================
  Files          31       31           
  Lines        2296     2296           
  Branches      191      191           
=======================================
  Hits         2054     2054           
  Misses        211      211           
  Partials       31       31           

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 0c68549...3ef5f3a. Read the comment docs.

@VKTB VKTB self-assigned this May 5, 2021
@MRichards99 MRichards99 merged commit f8b1136 into master May 5, 2021
@MRichards99 MRichards99 deleted the update-dependencies-#230 branch May 5, 2021 13:23
Copy link

@sam-glendenning sam-glendenning left a comment

Choose a reason for hiding this comment

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

Just about to approve this before it was merged! Looks good to me, gets rid of that annoying warning message.

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.

Update Dependencies
3 participants