| *** amorin_ is now known as amorin | 07:45 | |
| opendevreview | Rodolfo Alonso proposed openstack/requirements master: Remove ``sys_platform`` marker from ``pyroute2`` https://review.opendev.org/c/openstack/requirements/+/1008065 | 08:20 |
|---|---|---|
| *** bauzas67 is now known as bauzas | 08:38 | |
| opendevreview | Stephen Finucane proposed openstack/requirements master: ruff: Enable E5 rules https://review.opendev.org/c/openstack/requirements/+/1007742 | 09:41 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: ruff: Enable B, RUF rules https://review.opendev.org/c/openstack/requirements/+/1007743 | 09:47 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Ignore ruff-related patches https://review.opendev.org/c/openstack/requirements/+/1007744 | 09:47 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: tox: Fail if ruff format makes changes https://review.opendev.org/c/openstack/requirements/+/1000078 | 09:49 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: ruff: Enable E5 rules https://review.opendev.org/c/openstack/requirements/+/1007742 | 09:49 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: ruff: Enable B, RUF rules https://review.opendev.org/c/openstack/requirements/+/1007743 | 09:50 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Ignore ruff-related patches https://review.opendev.org/c/openstack/requirements/+/1007744 | 09:50 |
| stephenfin | frickler: mnasiadka: I had to respin https://review.opendev.org/c/openstack/requirements/+/1000078 because some new issues had crept in. Could we quickly send that through again before new ones get in? | 10:49 |
| stephenfin | tkajinam: Interestingly, it seems we have some users of sqlalchemy-utils but no one is declaring it in their requirements.txt file 😕 | 10:51 |
| stephenfin | https://codesearch.opendev.org/?q=sqlalchemy_utils (usage) vs. https://codesearch.opendev.org/?q=sqlalchemy-utils (requirement) | 10:51 |
| stephenfin | ah, wait, case sensitivity matters 🤦 https://codesearch.opendev.org/?q=SQLAlchemy-Utils&i=fosho | 11:03 |
| frickler | sadly https://github.com/kvesteri/sqlalchemy-utils/pull/831 was (very rudely IMHO) closed without a comment. so I think the future of that library should be considered questionable | 11:07 |
| stephenfin | agreed. our usage is minimal. I'd be inclined to drop it across the board | 11:07 |
| stephenfin | I have a change to drop it from taskflow already (where it has not been needed for years, since sqlalchemy proper has supported a JSON type for over a decade) | 11:08 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: ruff: Enable E5 rules https://review.opendev.org/c/openstack/requirements/+/1007742 | 13:29 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: ruff: Enable B, RUF rules https://review.opendev.org/c/openstack/requirements/+/1007743 | 13:29 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Ignore ruff-related patches https://review.opendev.org/c/openstack/requirements/+/1007744 | 13:29 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Remove sqlalchemy-filters https://review.opendev.org/c/openstack/requirements/+/1009048 | 13:38 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Remove SQLAlchemy-Utils https://review.opendev.org/c/openstack/requirements/+/1009049 | 13:38 |
| *** elodille1 is now known as elodilles | 14:23 | |
| opendevreview | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for sushy to new release 5.14.0 https://review.opendev.org/c/openstack/requirements/+/1009073 | 15:14 |
| opendevreview | Merged openstack/requirements master: tox: Fail if ruff format makes changes https://review.opendev.org/c/openstack/requirements/+/1000078 | 15:23 |
| opendevreview | Stephen Finucane proposed openstack/requirements master: Remove SQLAlchemy-Utils https://review.opendev.org/c/openstack/requirements/+/1009049 | 15:59 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!