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

✨ New PathValidationHelper class #341

Merged
merged 1 commit into from
Nov 19, 2023

Conversation

jrfnl
Copy link
Collaborator

@jrfnl jrfnl commented Nov 19, 2023

... to help validate paths provided via a ruleset property before using them in sniffs.

The utility method in this class will also convert the paths from relative paths to absolute paths. The resulting array will have the original relative path as the key and the final absolute path as the value to allow for lookups.

Includes dedicated unit tests.

... to help validate paths provided via a ruleset property before using them in sniffs.

The utility method in this class will also convert the paths from relative paths to absolute paths.
The resulting array will have the original relative path as the key and the final absolute path as the value to allow for lookups.

Includes dedicated unit tests.
@coveralls
Copy link

Coverage Status

coverage: 98.58% (+0.03%) from 98.554%
when pulling f02f160 on JRF/yoastcs-new-path-validation-helper
into bfb05c3 on develop.

@jrfnl jrfnl merged commit f17dfbe into develop Nov 19, 2023
25 checks passed
@jrfnl jrfnl deleted the JRF/yoastcs-new-path-validation-helper branch November 19, 2023 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants