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

[docs] Add notes about plugin removal from core #1251

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

andrewnicols
Copy link
Member

@andrewnicols andrewnicols commented Feb 3, 2025

This PR starts to formulate a process for removing plugins from core.

Copy link

netlify bot commented Feb 3, 2025

Deploy Preview for moodledevdocs ready!

Name Link
🔨 Latest commit 2d8f043
🔍 Latest deploy log https://app.netlify.com/sites/moodledevdocs/deploys/67a1aef1480e5c000893de39
😎 Deploy Preview https://deploy-preview-1251--moodledevdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@andrewnicols
Copy link
Member Author

Pinging @danmarsden

@andrewnicols andrewnicols force-pushed the removePluginFromCore branch 2 times, most recently from ea0cc7e to 80f9b3d Compare February 3, 2025 02:59
@mattporritt
Copy link
Collaborator

@andrewnicols
I think we need to be a bit clearer in the annoucment about the plugin being remove. We may want to say something that we will let the community know via the moodle.org forums and the tracker before a plugin is removed from core. And will endeavour where POSSIBLE to allow sufficent time for those using the plugin to understand the change and to update any workflows or dependencies.

@danmarsden
Copy link
Contributor

Thanks @andrewnicols - this looks much more sensible to me than the previous approach - I really like the statement "Plugins without a maintainer will not be accepted into the plugins database." and I suspect we should put this into the plugins db checklist as well (pinging @mudrd8mz to hear your thoughts too!)

@andrewnicols
Copy link
Member Author

Thanks both,

@mattporritt I've added to the docs to list of the specific places we aim to announce, and to put a time disclaimer in.

There are some cases where we won't announce but it probably isn't worth noting them. One such example is h5plib.

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.

3 participants