Skip to content

Conversation

carmenbianca
Copy link
Member

@carmenbianca carmenbianca commented Feb 8, 2023

Fixes #530

This was surprisingly annoying. I thought Poetry was supposed to make life easier 😩 I'm stuck using Poetry v1.1 to maintain poetry-core>=1.0.0 support, and it doesn't work quite as smoothly as later versions. Specifically, there was an endless loop for sphinxcontrib.jquery, which means I had to drop the RTD theme to preserve my patience for more important things.

@carmenbianca carmenbianca marked this pull request as draft February 8, 2023 19:58
@carmenbianca carmenbianca force-pushed the drop-3.6 branch 2 times, most recently from 48a58be to 3363cbd Compare February 8, 2023 21:39
@carmenbianca carmenbianca marked this pull request as ready for review February 8, 2023 21:39
@carmenbianca carmenbianca force-pushed the drop-3.6 branch 2 times, most recently from bef3a8a to 8b3fd05 Compare February 10, 2023 15:27
I've also removed the RTD theme dependency. For some reason, its
'sphinxcontrib.jquery' dependency caused an infinite loop in Poetry. I
spent far more time than I care for trying to fix this, so I've just
given up now. Alabaster is (or: has become) a decent theme.

Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
Let's try to be a little green and run fewer tests.

Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
Copy link
Member

@nicorikken nicorikken left a comment

Choose a reason for hiding this comment

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

Great work, thanks! This will enable other improvements.

@nicorikken
Copy link
Member

@carmenbianca in the future we'll get to reap more of the tooling benefits 🤞

@carmenbianca
Copy link
Member Author

Thanks @nicorikken ! we can already re-enable pytest coverage after this PR :)

@carmenbianca carmenbianca merged commit 0b0baef into fsfe:main Feb 17, 2023
carmenbianca added a commit to carmenbianca/reuse-tool that referenced this pull request Apr 7, 2023
Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
carmenbianca added a commit to carmenbianca/reuse-tool that referenced this pull request Apr 7, 2023
Signed-off-by: Carmen Bianca BAKKER <carmenbianca@fsfe.org>
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.

Drop support for Python 3.6
2 participants