Skip to content

Conversation

@mglaman
Copy link
Collaborator

@mglaman mglaman commented Jan 12, 2026

No description provided.

@mglaman mglaman requested a review from Copilot January 18, 2026 21:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the entire simplytest_messages custom Drupal module, which was previously used to display configurable warning messages on the site.

Changes:

  • Removed the complete simplytest_messages module including all source files, routing, configuration forms, and metadata

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
MessageBlock.php Removed the custom block plugin that displayed configurable messages
SettingsForm.php Removed the configuration form for managing message settings
simplytest_messages.routing.yml Removed routing configuration for the settings form
simplytest_messages.module Removed hook implementation for preprocessing the messages region
simplytest_messages.links.menu.yml Removed menu link for accessing the settings form
simplytest_messages.info.yml Removed module metadata file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mglaman
Copy link
Collaborator Author

mglaman commented Jan 18, 2026

@copilot fix PHPStan ignored errors that are causing the workflow to fail

Copy link

Copilot AI commented Jan 18, 2026

@mglaman I've opened a new pull request, #566, to work on those changes. Once the pull request is ready, I'll request review from you.

…566)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mglaman <3698644+mglaman@users.noreply.github.com>
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