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

Tracking list: external extensions broken by release 3.4 #1277

Closed
5 of 12 tasks
oprypin opened this issue Jul 18, 2022 · 11 comments
Closed
5 of 12 tasks

Tracking list: external extensions broken by release 3.4 #1277

oprypin opened this issue Jul 18, 2022 · 11 comments
Labels
wontfix The issue will not be fixed for the stated reasons.

Comments

@oprypin
Copy link
Contributor

oprypin commented Jul 18, 2022

@facelessuser
Copy link
Collaborator

Can ask what the purpose of this issue is? What is the actionable item being requested?

Hmm...I guess Python Markdown doesn't have discussions enabled...that would probably be a more appropriate place for this...

@oprypin
Copy link
Contributor Author

oprypin commented Jul 18, 2022

For tracking and for decision-making for myself and possibly for this repo too. I could've opened it on mkdocs repo but it's not about mkdocs.
It's nice as an issue because of GitHub's autolinking and formatting.

@oprypin
Copy link
Contributor Author

oprypin commented Jul 18, 2022

@tsibley
Copy link

tsibley commented Jul 18, 2022

@oprypin
Copy link
Contributor Author

oprypin commented Jul 19, 2022

OK, at this point I pretty much think that these breakages should not have happened in this way. The list of known breakages certainly already seems much bigger than expected.

I was trying to see how the community (mainly MkDocs) can deal with this, advocating for fixes, accepting to have to help users by pointing into the right place to report issues... But anyway I have been convinced that MkDocs can't take any reasonable step other than just preventing people from installing this newer version of Python-Markdown. Because indeed to non-technical users it will always just look like MkDocs is broken.

I was trying to argue that I don't want to just punt this into the future, but it seems like I'll have to do just that.


Can ask what the purpose of this issue is? What is the actionable item being requested?

I have to admit that part of the purpose was collecting data to be able to write this message...
And what's actionable is reconsidering the rollout of these breakages.

@waylan
Copy link
Member

waylan commented Jul 19, 2022

I haven't looked at the individual extensions to see what broke (nor do I have the time to go look through them all). However, I do know that the we removed support for a number of previously deprecated items in the 3.4 release. The thing is, those deprecations have been in place since version 3.0 released on 2018-09-21. In other words, the developers of the various (now broken) extensions have been ignoring deprecation warnings for the last ~3.75 years. I think we gave everyone plenty of warning.

That said, of there is anything we broke which is not related to those deprecations, please point them out.

@mitya57
Copy link
Collaborator

mitya57 commented Jul 19, 2022

What I have seen is related to deprecated items, yes. I agree that it was plenty of time.

@oprypin
Copy link
Contributor Author

oprypin commented Jul 19, 2022

the developers of the various (now broken) extensions have been ignoring deprecation warnings for the last ~3.75 years

I think the main issue is that there were actually no warnings whatsoever.

The website of MkDocs itself, which surely you must have updated within those last ~3.75 years, supposedly would've shown a warning every time it's built, because it has been relying on Python-Markdown/github-links (yes, within this same organization), which did not get updated until I pointed it out yesterday.

@mitya57
Copy link
Collaborator

mitya57 commented Jul 19, 2022

Problem is that Python does not show DeprecationWarnings by default, unless you enable development mode.

@waylan
Copy link
Member

waylan commented Jul 19, 2022

@mitya57 is correct. The warnings were being issued, it's just that by default Python silenced them. Unfortunately there is nothing we can do to change that. Each user/developer needs to set up their own system to alert them of such warnings.

First of all, all of the deprecations were clearly identified in the release notes, so they should be of no surprise to anyone who reads them. I had intended to remove the code deprecated in 3.0 in 3.1 but held off because I recognized that most developers hadn't updated their extensions yet. In fact, in #1260 were the code was removed, I questioned whether we might want to raise some sort of Deprecation Error because I recognized that it was very likely that this would break many extensions. At least an error would clue them into why. But the decision was made to not do so as that would be unconventional.

@oprypin I'm sure you are not suggesting that we never remove deprecated code because Python has set a default we don't agree with. After all, that would be ridiculous. I understand you are frustrated, but we have been more than accommodating by waiting 3.75 years and 4 point releases to remove the deprecated code.

Again, if there are any issues which are not related to the long overdue removal of deprecated code, I am open to discuss them in a new issue. However, we have done everything we reasonably can to warn people of the coming deprecations. Therefore, I am closing this issue as it is not actionable. That said, if anyone has constructive suggestions for any ways to improve in how we communicate deprecations in the future, please feel free to continue the discussion here. However, arguments over past decisions will result in a user either being banned and/or the conversation being locked as the Python-Markdown development team deems appropriate.

@stickgrinder
Copy link

Don't know if it's worth commenting here about this, but I just sent this PR which fixes the problem for fontawesome-markdown.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix The issue will not be fixed for the stated reasons.
Projects
None yet
Development

No branches or pull requests

6 participants