- π¬π§United Kingdom catch
It's been more than a few days, so I'm going to go ahead and remove the tag. We have warnings for deprecated modules, so if an experimental module ends up on its way out of core, people will be notified that way. The info level item is still a reminder you're running something that's not fully stable yet.
- πΊπΈUnited States xjm
Let's bring it up in the RM channel so that folks have a chance to discuss and agree.
- π¬π§United Kingdom catch
Could use screenshots of the info-level message in the status report. The test coverage asserts that the warning doesn't appear, but should that change to a positive assertion that an info appears?
- π³πΏNew Zealand quietone New Zealand
xjm β credited quietone β .
- πΊπΈUnited States xjm
Via @quietone:
+++ b/core/modules/system/system.install @@ -98,7 +98,7 @@ function system_requirements($phase) { $requirements['experimental_modules'] = [ 'title' => t('Experimental modules enabled'), 'value' => t('Experimental modules found: %module_list. <a href=":url">Experimental modules</a> are provided for testing purposes only. Use at your own risk.', ['%module_list' => implode(', ', $experimental_modules), ':url' => 'https://www.drupal.org/core/experimental']), - 'severity' => REQUIREMENT_WARNING, + 'severity' => REQUIREMENT_INFO, ]; } // Warn if any deprecated modules are installed. @@ -147,7 +147,7 @@ function system_requirements($phase) { @@ -147,7 +147,7 @@ function system_requirements($phase) { $requirements['experimental_themes'] = [ 'title' => t('Experimental themes enabled'), 'value' => t('Experimental themes found: %theme_list. Experimental themes are provided for testing purposes only. Use at your own risk.', ['%theme_list' => implode(', ', $experimental_themes)]), - 'severity' => REQUIREMENT_WARNING, + 'severity' => REQUIREMENT_INFO, ];
This patch is going too far. The scope should be limited to runtime warnings being turned to info mesages. We should still be warning about them during installation/update.
I am also not sure the warning should be removed from the help pages.
- Status changed to Needs work
over 1 year ago 3:17pm 10 February 2023