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

Fix save passwords extra #53

Merged
merged 2 commits into from
Nov 18, 2023
Merged

Conversation

PeterJCLaw
Copy link
Member

@PeterJCLaw PeterJCLaw commented Jan 14, 2023

Builds on #55 to avoid other build system issues.

This is a bit of a hack though it fixes CI for now. It's not clear
why letting setuptools install Sphinx as part of the `install_requires`
dependencies doesn't work, however as we're likely going to need
to rework how installs work soon anyway (since python packaging
as a whole is shifting) it doesn't seem worth spending too much
time on fixing it right now.
@PeterJCLaw PeterJCLaw force-pushed the fix-save-passwords-extra branch from b8aa6c4 to b24bce8 Compare June 14, 2023 21:43
I can't find a spelling where this worked previously (everything
gives a syntax error or doesn't match), so I'm not worried about
this being a breaking change as such. In any case the next release
will be a major version bump.
@PeterJCLaw PeterJCLaw force-pushed the fix-save-passwords-extra branch from b24bce8 to 8d7e341 Compare June 14, 2023 21:43
@PeterJCLaw PeterJCLaw marked this pull request as ready for review June 14, 2023 21:47
@PeterJCLaw PeterJCLaw merged commit 700cace into srobo:main Nov 18, 2023
@PeterJCLaw PeterJCLaw deleted the fix-save-passwords-extra branch November 18, 2023 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant