Problem/Motivation

Some missing documentation: https://git.drupalcode.org/project/config_readonly_taxonomy_ui/-/jobs/10...

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork config_readonly_taxonomy_ui-3601426

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

prudloff created an issue. See original summary.

nakaza sora’s picture

nakaza sora commented
Assigned: Unassigned » nakaza sora

nakaza sora opened merge request !1

nakaza sora’s picture

nakaza sora commented
Assigned: nakaza sora » Unassigned
Status: Active » Needs review

Created a merge request that fixes the phpcs (Drupal & DrupalPractice) errors reported by CI:

- src/Form/VocabularyForm.php: added the missing class and method docblocks.
- config_readonly_taxonomy_ui.info.yml: namespaced the dependencies (drupal:taxonomy, config_readonly:config_readonly).

Coding standards only, no behavior change. phpcs now passes with no errors.

Could you please review it when you have time? Thank you!

vinodhini.e’s picture

vinodhini.e commented

Hi, Tested this on Drupal 11.3.11 and encountered the same PHPCS coding standard issues while running the PHPCS checks.

Applied Merge #1 and re-ran PHPCS:

The previously reported documentation-related errors are no longer appearing, and the PHPCS checks are now passing successfully.

Thanks.

prudloff’s picture

prudloff commented
Status: Needs review » Fixed

Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.