Skip to content

DEVX-5995 PR codacy fixes#100

Open
namespacebrian wants to merge 5 commits intomainfrom
devx-5995-claude
Open

DEVX-5995 PR codacy fixes#100
namespacebrian wants to merge 5 commits intomainfrom
devx-5995-claude

Conversation

@namespacebrian
Copy link
Contributor

No description provided.

kporras07 and others added 4 commits November 12, 2025 16:44
- Add missing @param documentation for $options parameter
- Add missing @return tag for connectionLink method
- Add detailed parameter descriptions for determineSourceAndVcsOrg method
- Add detailed parameter descriptions for findVcsOrgInPantheonOrg method

Addresses 7 Codacy documentation issues (lines 23, 49, 126-129, 189-190).
The WordPress security rules for output escaping don't apply to this CLI
application. These rules were causing false positives for TerminusException
error messages which use safe placeholder interpolation.

Disables:
- WordPress.Security.EscapeOutput.OutputNotEscaped
- WordPress.XSS.EscapeOutput.OutputNotEscaped
@namespacebrian namespacebrian requested a review from a team as a code owner November 13, 2025 19:47
@namespacebrian namespacebrian changed the base branch from devx-5995 to main November 13, 2025 19:48
Disables Squiz.Commenting.FunctionComment spacing rules that enforce
excessive whitespace alignment in parameter documentation. These rules
require exact spacing alignment that doesn't improve code readability.

Disabled rules:
- Squiz.Commenting.FunctionComment.SpacingAfterParamType
- Squiz.Commenting.FunctionComment.SpacingAfterParamName

This should resolve the remaining Codacy spacing complaints while
maintaining readable and consistent documentation.
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