-
Notifications
You must be signed in to change notification settings - Fork 2k
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
dev-python/humanfriendly: bump to 4.17 #10452
Conversation
Package-Manager: Portage-2.3.51, Repoman-2.3.12 Signed-off-by: Tomáš Mózes <hydrapolic@gmail.com>
Copyright policy changePlease note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article. Pull Request assignmentAreas affected: ebuilds dev-python/humanfriendly: @gentoo/python Linked bugsNo bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment. If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers. In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
Pull request CI reportReport generated at: 2018-11-20 07:37 UTC No issues found |
Package-Manager: Portage-2.3.51, Repoman-2.3.12 Signed-off-by: Tomáš Mózes <hydrapolic@gmail.com> Closes: gentoo#10452 Signed-off-by: Mikle Kolyada <zlogene@gentoo.org>
>=dev-python/capturer-2.1[${PYTHON_USEDEP}] | ||
>=dev-python/coloredlogs-2.0[${PYTHON_USEDEP}] | ||
>=dev-python/pytest-3.0.7[${PYTHON_USEDEP}] | ||
>=dev-python/pytest-2.4.0[${PYTHON_USEDEP}] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a typo? (pytest required twice)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, it should have been pytest-3.0.7 and pytest-cov-2.4.0, i'll correct, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
S="${WORKDIR}/${MY_P}" | ||
|
||
python_test() { | ||
pytest -v || die |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't -vv
the correct form? I know this got merged already, but you figure the answer to these questions when bumping the next time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't see a difference in the output of pytest -v and pytest -vv. Looking at the packages, some packages have only pytest, some pytest -v and some pytest -vv.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to https://wiki.gentoo.org/wiki/Project:Python/Tests it's a single -v.
Tests pass on ~amd64.