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

Hotfix: New rector FILE_STATUS_PERMANENT was not added to deprecation list. … #280

Merged
merged 2 commits into from
Dec 28, 2023

Conversation

bbrala
Copy link
Collaborator

@bbrala bbrala commented Dec 28, 2023

…Only config object was intialized.

Description

Explain the technical implementation of the work done.

To Test

  • Add steps to test this feature

Drupal.org issue

Provide a link to the issue from https://www.drupal.org/project/rector/issues. If no issue exists, please create one and link to this PR.

@bbrala bbrala requested a review from agentrickard December 28, 2023 08:34
@@ -61,7 +61,7 @@ class DBRector extends AbstractRector implements ConfigurableRectorInterface
private array $configuration;

/**
* @var \DrupalRector\Services\AddCommentService
* @var AddCommentService
Copy link
Contributor

Choose a reason for hiding this comment

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

My understanding is that @var descriptions should use the full class path, so I don't understand these changes.

Copy link
Contributor

Choose a reason for hiding this comment

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

@@ -61,7 +61,7 @@ class DBRector extends AbstractRector implements ConfigurableRectorInterface
private array $configuration;

/**
* @var \DrupalRector\Services\AddCommentService
* @var AddCommentService
Copy link
Contributor

Choose a reason for hiding this comment

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

@agentrickard agentrickard merged commit 94650bc into main Dec 28, 2023
7 checks passed
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