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

dev-python/flask-paginate: bumped to 0.5.5 #16800

Closed
wants to merge 3 commits into from
Closed

dev-python/flask-paginate: bumped to 0.5.5 #16800

wants to merge 3 commits into from

Conversation

florianjr
Copy link
Contributor

Signed-off-by: Florian Förster foerster.florianjr@gmail.com
Closes: https://bugs.gentoo.org/717526
Package-Manager: Portage-3.0.0, Repoman-2.3.23

Signed-off-by: Florian Förster <foerster.florianjr@gmail.com>
Closes: https://bugs.gentoo.org/717526
Package-Manager: Portage-3.0.0, Repoman-2.3.23
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @florianjr
Areas affected: ebuilds
Packages affected: dev-python/flask-paginate

dev-python/flask-paginate: @gentoo/proxy-maint, @stkw0, @gentoo/python

Linked bugs

Bugs linked: 717526


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Jul 24, 2020
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-07-24 16:06 UTC
Newest commit scanned: 8ca50dd
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/126ee60d76/output.html

@florianjr
Copy link
Contributor Author

Could I help maintaining this package furthermore?

@stkw0
Copy link
Contributor

stkw0 commented Jul 25, 2020

Could I help maintaining this package furthermore?

I will be glad if you maintain this, maybe you can add yourself to the maintainers list. :)

@florianjr
Copy link
Contributor Author

How can I do this? :)

@stkw0
Copy link
Contributor

stkw0 commented Jul 25, 2020

There is a file called metadata.xml with a list of maintainers. But I think before that a Gentoo developer should register you as a maintainer.

@florianjr
Copy link
Contributor Author

Who should I contact? Then I can push it again with the right metadata.

@florianjr
Copy link
Contributor Author

You mean as a proxy-maintainer?

@stkw0
Copy link
Contributor

stkw0 commented Jul 26, 2020

You mean as a proxy-maintainer?

Yes. There is an IRC channel where you can ask.

Package-Manager: Portage-3.0.0, Repoman-2.3.23
@florianjr
Copy link
Contributor Author

Here i added myself to the maintainers list, i think this should be enough.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-07-26 07:31 UTC
Newest commit scanned: c1426bc
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/68ea92abe8/output.html

@florianjr
Copy link
Contributor Author

@stkw0 is this now enough to be merged?

@stkw0
Copy link
Contributor

stkw0 commented Jul 27, 2020

@stkw0 is this now enough to be merged?

Version 0.7.0 is available, maybe we should bump to that version instead of 0.5.5. Also seems you forgot to update MY_COMMIT variable? It seems to be the same commit than in the older version.

@florianjr
Copy link
Contributor Author

I’m kinda newbie, could you explain me the difference between $MY_COMMIT and $PV? I’ll do the bump later. Thanks for your attention :)

Package-Manager: Portage-3.0.0, Repoman-2.3.23
@florianjr
Copy link
Contributor Author

I’ve now bumped it again to newest version (0.6.0)

DESCRIPTION="Pagination support for flask"
HOMEPAGE="https://flask-paginate.readthedocs.io"
# https://github.com/lixxu/flask-paginate/issues/68
SRC_URI="https://github.com/lixxu/flask-paginate/archive/${MY_COMMIT}.tar.gz -> ${P}.tar.gz"
Copy link
Member

Choose a reason for hiding this comment

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

0.7.0 is the latest version, and this is not correct, you should use SRC_URI="https://github.com/lixxu/flask-paginate/archive/${PV}.tar.gz"

Copy link
Contributor

@stkw0 stkw0 Jul 28, 2020

Choose a reason for hiding this comment

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

Please also remove that comment line in new ebuilds. Now that the issue have been fixed it only disturbs :)

Thank you for your effort.


inherit distutils-r1

MY_COMMIT="510ad833106134711868653fb597bf75ea8ac34f"
Copy link
Member

Choose a reason for hiding this comment

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

This commit is from 0.5.3 version (lixxu/flask-paginate@510ad83)

@florianjr
Copy link
Contributor Author

Ohhhh sorry. I’ll open a new and “clean” PR later, thanks for your help :)

@stkw0
Copy link
Contributor

stkw0 commented Jul 28, 2020

Ohhhh sorry. I’ll open a new and “clean” PR later, thanks for your help :)

You don't need to do that. git rebase can do magic to fixup commits, remove commits and so ;)
Check out https://wiki.gentoo.org/wiki/Gentoo_GitHub and more specifically https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History

@florianjr florianjr closed this Jul 28, 2020
@florianjr florianjr reopened this Jul 28, 2020
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-07-28 20:51 UTC
Newest commit scanned: 8673fd1
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/0f3e5b7a6f/output.html

@florianjr florianjr closed this Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR.
Projects
None yet
5 participants