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

fix: tag removal for 0-line tags (if other tags present, only blank out the tag; otherwise, remove to end of block) #1041

Merged
merged 1 commit into from
Apr 24, 2023

Conversation

brettz9
Copy link
Collaborator

@brettz9 brettz9 commented Apr 21, 2023

fixes #1040

@JoshuaKGoldberg : Could you take a look at the test cases and see if they are as you would expect? I find it a little strange that we are wiping out whole tags including their descriptions unless only one is left, in which case, we leave one description. Should we not wipe it out entirely if that is the goal of the option?

…ut the tag; otherwise, remove to end of block); fixes gajus#1040
@brettz9 brettz9 force-pushed the check-tag-names-typed branch from 58178a1 to e99f4e8 Compare April 24, 2023 16:36
@brettz9 brettz9 merged commit 4f3c48e into gajus:main Apr 24, 2023
@brettz9
Copy link
Collaborator Author

brettz9 commented Apr 24, 2023

@JoshuaKGoldberg : I figured it is better to have a fix now, but if you could look at the test cases in this PR, it would help make sure the fixing behavior is as intended.

@brettz9 brettz9 deleted the check-tag-names-typed branch April 24, 2023 22:02
@github-actions
Copy link

🎉 This PR is included in version 43.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@JoshuaKGoldberg
Copy link
Contributor

Hi sorry for the delay - test cases look great to me! Thank you for fixing this so quickly! 🙏

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.

Autofix breaks comments removing zero/final line
2 participants