-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update core dependencies and remove final traces of the Authlist feature #270
Conversation
c81a14f
to
bae3621
Compare
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.
LGTM. I left a few comments and questions for your consideration.
@@ -1,10 +1,9 @@ | |||
FROM python:3.8.2-buster | |||
FROM python:3.11-bookworm |
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.
Are there plans to migrate the Dockerfiles to AlmaLinux9? bookworm
sounds like a Debian derivative. TSSW uses AlmaLinux with Mamabaforge for Python and conda.
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.
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.
Thanks for the info!
CHANGELOG.rst
Outdated
v7.0.0 | ||
------ | ||
|
||
* Update core dependencies `<https://github.com/lsst-ts/LOVE-manager/pull/270>`_ |
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.
Looks like more happened, like remove Authlist support. Not sure if you want to mention that here as well?
bae3621
to
0b36806
Compare
0b36806
to
9e38a3b
Compare
This PR makes several updates on the dependencies of the LOVE-manager application. A set of the ones with major updates is:
Also adjustments for deprecated features were done.
Finally, a couple traces of the Authlist feature were removed too. This was causing bunch of warnings on the startup of the service.