Skip to content

Add Compat class for aliasing core MediaWiki classes#21

Open
WikiMANNia wants to merge 1 commit intomywikis:mainfrom
WikiMANNia:patch-2
Open

Add Compat class for aliasing core MediaWiki classes#21
WikiMANNia wants to merge 1 commit intomywikis:mainfrom
WikiMANNia:patch-2

Conversation

@WikiMANNia
Copy link
Copy Markdown

No description provided.

@jeffw16
Copy link
Copy Markdown
Member

jeffw16 commented Mar 29, 2026

Thanks for this proposed change.

It's not clear to me why this is necessary though; would you be able to elaborate?

To ensure an atomic update, would you please combine all file changes for the same feature into one pull request?

Please note that, as of 1.2.0, MediaWiki 1.35 is no longer supported due to the difficulty of running the workflows to test it.

@WikiMANNia
Copy link
Copy Markdown
Author

This way to manage backward compatibility will no longer disturbing code

@WikiMANNia
Copy link
Copy Markdown
Author

WikiMANNia commented Mar 29, 2026

To ensure an atomic update, would you please combine all file changes for the same feature into one pull request?

How can I do it?

@jeffw16
Copy link
Copy Markdown
Member

jeffw16 commented Mar 29, 2026

This way to manage backward compatibility will no longer disturbing code

Sorry, I don't understand what this means…

To ensure an atomic update, would you please combine all file changes for the same feature into one pull request?

How can I do it?

You can go to this patch's branch (https://github.com/WikiMANNia/Mediawiki-Extension-CrawlerProtection/tree/patch-2) and make the changes in there. They'll automatically be reflected in this PR.

@WikiMANNia
Copy link
Copy Markdown
Author

You can go to this patch's branch (https://github.com/WikiMANNia/Mediawiki-Extension-CrawlerProtection/tree/patch-2) and make the changes in there.

I didn't see it. Thank for indicating me!

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.

2 participants