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

test: safety check #924

Merged
merged 1 commit into from
Apr 15, 2020
Merged

test: safety check #924

merged 1 commit into from
Apr 15, 2020

Conversation

rerowep
Copy link
Contributor

@rerowep rerowep commented Apr 14, 2020

  • Replaces pipenv check with pipenv run safety check because pipenv
    was not working any more.

Co-Authored-by: Peter Weber peter.weber@rero.ch

Why are you opening this PR?

  • pipenv check was not working any more.

How to test?

  • run ./run-tests.sh

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

@rerowep rerowep self-assigned this Apr 14, 2020
@rerowep rerowep requested review from blankoworld, jma and BadrAly April 14, 2020 09:42
Pipfile.lock Outdated Show resolved Hide resolved
* Replaces `pipenv check` with `pipenv run safety check` because pipenv
was not working any more.

Co-Authored-by: Peter Weber <peter.weber@rero.ch>
@rerowep rerowep requested review from lauren-d and zannkukai April 14, 2020 12:58
@rerowep rerowep merged commit b8faefb into rero:dev Apr 15, 2020
@rerowep rerowep deleted the wep-pyup branch April 15, 2020 08:07
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.

3 participants