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

updated xstatic-font-awesome #1326

Closed
ReimarBauer opened this issue Dec 22, 2022 · 39 comments
Closed

updated xstatic-font-awesome #1326

ReimarBauer opened this issue Dec 22, 2022 · 39 comments
Labels

Comments

@ReimarBauer
Copy link
Member

Hi,
I had to update a xstatic package, and added there a toml definition too.
Take it as an example if needed
https://github.com/ReimarBauer/xstatic-font-awesome

greetings
Reimar

@ReimarBauer
Copy link
Member Author

@ThomasWaldmann could you create from this a new pypi package? Or add me as maintainer.

@ThomasWaldmann
Copy link
Member

@ReimarBauer Sure, I just invited you to the pypi project, so you can help. Thanks!

@ThomasWaldmann
Copy link
Member

ThomasWaldmann commented Dec 22, 2022

https://github.com/xstatic-py

https://github.com/python-xstatic

Not sure why I created 2 orgs...

And there is also the stuff at https://github.com/openstack ...

@ReimarBauer @deshipu guess we should only have 1 org. any preference about which one?

@ReimarBauer
Copy link
Member Author

ReimarBauer commented Dec 22, 2022

I asked first in the openstack because I thought it is an easy task openstack-archive/xstatic-font-awesome#2 not knowing that the one I asked don't be responsible. He mentioned there are more maintainers.

there it is
https://pypi.org/project/XStatic-Font-Awesome/6.2.1.0/

This has compared to the old one a lot more data and I do use /static/font_awesome/data/css/all.css or the min package.

There was also a name change fontawesome.css the old one was font-awesome.css

I would prefer https://github.com/xstatic-py

We should also think about if we can copy some ideas from conda-forge, especially the one which creates pull requests for the recipe when the source gets a new release.

I find it useful for the packages not to establish a complex collaboration processes, since we are only building a meta layer, the code is from someone else.

@deshipu
Copy link
Contributor

deshipu commented Jan 11, 2023

@ThomasWaldmann sorry for the delay, I was on holidays. There seems to be a bit more of a mess with this. Since some of the xstatic packages that we used in Horizon seemed to be unmaintained, and there were no repositories on github for them, we adopted them into the openstack organization. But suddenly you are making a release, with changes that break horizon, in the middle of holidays, ignoring the openstack release process. That got some people concerned. So I would love to clarify the ownership of those packages, as well as the correct repositories for them (the pypi page lists the fontawesome repository as the project's repository, which seems wrong).

@ThomasWaldmann
Copy link
Member

ThomasWaldmann commented Jan 11, 2023

@deshipu sorry if i broke something (what package/what release was it?). I wasn't aware of any "openstack release process".

If horizon has very strict requirements and installs dependencies automatically, it maybe make sense if you pin versions there, so that newer versions only get installed after some developer tested them.

@deshipu
Copy link
Contributor

deshipu commented Jan 11, 2023

No harm done, it just made some people worried. We pinned the version and all is good. It was the font-awesome one. If it's going to live under the xstatic repository, I'm going to ask to remove it form under the openstack care. If possible, I would like to review the other xstatic packages, to make sure everyone are on the same page about where they belong. I will prepare a list tomorrow.

@ReimarBauer
Copy link
Member Author

ReimarBauer commented Jan 11, 2023

@deshipu me was that culprit. I did run in the problem that this and some other packages seems not been updated since long. This version was required essential for me. That gh was used https://github.com/ReimarBauer/xstatic-font-awesome

I also don't pin all packages, sometimes it would help, sometimes the problem arises in some underlaying library. We had also trouble over the holidays.

We should get to a state that packages can get updates.

@deshipu
Copy link
Contributor

deshipu commented Jan 11, 2023

Let's just clarify where everything belongs, and add proper links to the correct repositories on pypi, this should resolve most of the problems. We will make sure that only openstackci is a maintainer on those projects that remain under OpenStack, so that the release process will be enforced on those.

@ReimarBauer
Copy link
Member Author

just looked into the web interface and have not seen an option to change links there. I do tommorrow a new upload, and remove that tox.ini which was leftover.

@deshipu
Copy link
Contributor

deshipu commented Jan 11, 2023

I think it's defined in the project.toml:

https://github.com/ReimarBauer/xstatic-font-awesome/blob/master/pyproject.toml#L27-L30

@ReimarBauer
Copy link
Member Author

yes, and that will update the pypi server on a new build there. In the past I think we could alter meta data also in the webfrontend.

@ReimarBauer
Copy link
Member Author

That repository entry is neat, compared to some other older packages, e.g.

https://pypi.org/project/XStatic-Pygments/

Once you accept that you need tomls that change is a good improvement.

@deshipu
Copy link
Contributor

deshipu commented Jan 12, 2023

Here is the list of xstatic packages that got adopted into the OpenStack repositories:

  • xstatic-font-awesome
  • xstatic-angular
  • xstatic-jquery-migrate
  • xstatic-moment-timezone
  • xstatic-dagre-d3
  • xstatic-moment
  • xstatic-lodash
  • xstatic-dagre
  • xstatic-graphlib
  • xstatic-json2yaml
  • xstatic-js-yaml
  • xstatic-filesaver
  • xstatic-angular-vis
  • xstatic-angular-uuid
  • xstatic-angular-notify
  • xstatic-angular-material
  • xstatic-roboto-fontface
  • xstatic-mdi
  • xstatic-bootswatch
  • xstatic-angular-gettext
  • xstatic-magic-search
  • xstatic-angular-lrdragndrop
  • xstatic-angular-smart-table
  • xstatic-angular-bootstrap
  • xstatic-spin
  • xstatic-rickshaw
  • xstatic-hogan
  • xstatic-d3
  • xstatic-bootstrap-scss
  • xstatic-bootstrap-datepicker
  • xstatic-jsencrypt
  • xstatic-jquery.tablesorter
  • xstatic-jquery.quicksearch
  • xstatic-jasmine

If you would like to maintain any of them outside of OpenStack, we can remove them from our repositories. For the ones that remain, I would like to remove any maintainers other than openstackci from their PyPi pages, so that releases can only be done through the OpenStack release process. Would that work for you? What do you think?

@ReimarBauer
Copy link
Member Author

As long as a selection is only used by OpenStack, no problem occurs.

How do we handle this if someone needs an update "tomorrow"? Can we then successively move the package?
In that case OpenStack would have to do a pinning.
Or what is the process for this?

The reverse is the same if a single maintainer does not respond to an upgrade request.

I would like to put everything I do for my projects in a github organization.

My pinning looks like this now

XStatic-jQuery==3.5.1.1
XStatic-Font-Awesome>=6.2.1.0
XStatic-jquery-ui==1.13.0.1
XStatic-Bootstrap==4.5.3.1
XStatic-Select2==4.0.13.1

@ReimarBauer
Copy link
Member Author

MoinMoin has
'XStatic>=0.0.2', # support for static file pypi packages
'XStatic-Bootstrap==3.1.1.2',
'XStatic-Font-Awesome>=4.1.0.0',
'XStatic-CKEditor>=3.6.1.2',
'XStatic-autosize',
'XStatic-jQuery>=1.8.2',
'XStatic-jQuery-File-Upload>=10.31.0',
'XStatic-TWikiDraw-moin>=2004.10.23.2',
'XStatic-AnyWikiDraw>=0.14.2',
'XStatic-svg-edit-moin>=2012.11.15.1',
'XStatic-JQuery.TableSorter>=2.14.5.1',
'XStatic-Pygments>=1.6.0.1',

@deshipu
Copy link
Contributor

deshipu commented Jan 12, 2023

So it looks like only xstatic-font-awesome and xstatic-jquery.tablesorter are shared. Would the MoinMoin project be happy to maintain those two packages, following its own contribution guidelines? We can then remove them from our repositories.

As for OpenStack contribution process, it's described in excruciating detail at https://docs.openstack.org/contributors/code-and-documentation/index.html, but in short you submit a patch to the project's gerrit, where it is reviewed by other developers and merged if it gets at least two +2 votes. If a version number was incremented, a release is made automatically by our continuous integration scripts. Only members of the project's core developers team can give +2 votes, and new members are added by the existing members based on contributions.

@ReimarBauer
Copy link
Member Author

I do agree to this. @ThomasWaldmann and @RogerHaase fine for you too?

@RogerHaase
Copy link
Member

RogerHaase commented Jan 12, 2023 via email

@ThomasWaldmann
Copy link
Member

guess i will rarely contribute, so choose whatever works best for you all.

just as a side note: compared to the "ease of use" and widespreadness of github (people having accounts, familiarity with PR process, etc), IMHO that openstack contribution process might be rather repelling (or at least not attracting) outside contributions.

RogerHaase added a commit to RogerHaase/moin that referenced this issue Jan 24, 2023
RogerHaase added a commit that referenced this issue Jan 24, 2023
Update FontAwesome macro and moinwiki rst docs for FontAwesome 6, #1326
@manchandavishal
Copy link

Hi @ReimarBauer hope you are doing well, I was about to retire the "xstatic-font-awesome" repo from OpenStack tree but Can you please clarify the URL for xstatic-font-awesome and xstatic-jquery.tablesorter repos where you are going to maintain it. I will update the same URL in README.txt file https://github.com/openstack/xstatic-font-awesome/blob/master/README.txt which OpenStack contributor refers to if they want to contribute. Should I mention https://github.com/xstatic-py/xstatic-font-awesome font-awesome ? what will be the URL for xstatic-jquery.tablesorter repo?

@manchandavishal
Copy link

@ReimarBauer One correction here, want to know about Xstatic-jquery-ui repo URL not xstatic-jquery.tablesorter.

@ReimarBauer
Copy link
Member Author

Hi @manchandavishal, mostly fine. Currently fixing things after setuptools changes ;)

xstatic-font-awesome, this will get soon a new build, we take care for it in https://github.com/xstatic-py/xstatic-font-awesome.
In further versions we can show the repository url as in the new build. That is much easier to stay in contact.

Also the other one will be done in the https://github.com/xstatic-py organisation.
see https://github.com/xstatic-py/xstatic-jquery_ui

I will not do this packaging community work in a user repo.

@manchandavishal
Copy link

@ReimarBauer ok, gotcha. thanks for the quick response.

@ReimarBauer
Copy link
Member Author

Btw. do you be on FOSDEM meetup?

@manchandavishal
Copy link

No.

@ReimarBauer
Copy link
Member Author

So here we are :)
https://pypi.org/project/XStatic-Font-Awesome/6.2.1.1/

and first time we have a xstatic- repository link in the upload.

new_xstatic-repository_link

@deshipu
Copy link
Contributor

deshipu commented Jan 31, 2023

What about the xstatic-jquery.tablesorter? Or are you happy for this to stay at OpenStack? MoinMoin uses it, though, so maybe still want to have it under your control?

@ReimarBauer
Copy link
Member Author

Lets have a look on the state of that.

in 2020 there was v2.31.3 the last release https://github.com/Mottie/tablesorter/tags
https://plugins.jquery.com/tablesorter/
on pypi we have currently 2.14.5.2

It may need more people to help
https://github.com/Mottie/tablesorter/issues

It blocks currently support for JQuery 3.6.0
Mottie/tablesorter#1823

When someone at OpenStack does a v2.31.3 xstatic package I fear this will stay for long. Or we find someone who refactors it for JQuery 3.6.0.

So yes I am happy it stays at OpenStack.

@manchandavishal
Copy link

@ReimarBauer are you ok with maintaining "xstatic-jquery" like you are doing for xstatic-jquery_ui and xstatic-font-awesome ?
Please let me know?

@ReimarBauer
Copy link
Member Author

@manchandavishal I do. This is also our example for XStatic itselfs. This will also get the repository url.

@manchandavishal
Copy link

@ReimarBauer ok thanks for confirming, we are retiring xstatic-jquery_ui, xstatic-font-awesome and xstatic-jquery packages from OpenStack and going to use them from https://github.com/xstatic-py.

@ReimarBauer
Copy link
Member Author

@manchandavishal when you update, or do new packages would you so kind also add this repository url? I think this is much easier to see for new users how and who is helping with these packages.

@manchandavishal
Copy link

@ReimarBauer yeah I will update repository URL in README.txt file like done here https://review.opendev.org/c/openstack/xstatic-font-awesome/+/872106/2/README.txt

@ReimarBauer
Copy link
Member Author

@deshipu What was wrong with me as maintainer of that package https://pypi.org/project/XStatic-Font-Awesome/ ?

@ReimarBauer
Copy link
Member Author

@manchandavishal I thought we agreed to maintain this package under the xtatic-py organization, just got a mail that I was removed from https://pypi.org/project/XStatic-Font-Awesome/
Mistake?

@deshipu
Copy link
Contributor

deshipu commented Apr 5, 2023

Ouch, that is my mistake. I will work on putting the maintainers back, sorry about this. I was removing the maintainers from other pypi projects, and included this project by mistake.

@deshipu
Copy link
Contributor

deshipu commented Apr 5, 2023

I put everyone back, sorry for the problem again.

@ReimarBauer
Copy link
Member Author

@ThomasWaldmann @deshipu do you have interest do become a maintainer role for that package on conda-forge too?
conda-forge/staged-recipes#22478

I currently prepare that.

openstack-mirroring pushed a commit to openstack/governance that referenced this issue Apr 14, 2023
xstatic-angular-fileupload pypi is used by Horizon.
Horizon team discussed this topic in the horizon weekly
meeting and decided to move it under the horizon deliverables
and keep them under the OpenStack namespace [1]. We also check
with moinwiki contributor and they are not using this pypi project [2].

[1] https://meetings.opendev.org/irclogs/%23openstack-horizon/%23openstack-horizon.2023-02-01.log.html#t2023-02-01T16:09:57
[2] moinwiki/moin#1326 (comment)

Needed-By: https://review.opendev.org/c/openstack/project-config/+/873843
Change-Id: I8a1c1afdad7f2768e80c1efc5cdbcd914abc8e71
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants