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

Support "//" comments in AttributeAndTargetSpacingSniff #1497

Closed
wants to merge 1 commit into from

Conversation

mmoll
Copy link
Contributor

@mmoll mmoll commented Jan 16, 2023

No description provided.

@kukulich
Copy link
Contributor

// is not doc comment so it's not valid for this sniff.

@mmoll mmoll changed the title Support "//" comments in RequireAttributeAfterDocCommentSniff Support "//" comments in AttributeAndTargetSpacingSniff Jan 16, 2023
@mmoll
Copy link
Contributor Author

mmoll commented Jan 16, 2023

// is not doc comment so it's not valid for this sniff.

Alright, now I found out, that the problem is indeed in a different sniff, but at the moment I'm a bit lost how to solve this, apart from the failing test.

@kukulich kukulich closed this Feb 1, 2023
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