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: don't include source with issue URL comment #241

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

rgalonso
Copy link
Contributor

If TODO comment is a suffix to a line of
(executable) source, don't repeat the source
content when inserting the issue URL. But be sure
to still keep the same alignment.

Closes #229

If TODO comment is a suffix to a line of
(executable) source, don't repeat the source
content when inserting the issue URL. But be sure
to still keep the same alignment.

Closes alstr#229
@rgalonso
Copy link
Contributor Author

@alstr, hopefully this is the last PR for a while. At the very least we should be done with the big refactors for now. Ready for your review.

@alstr alstr merged commit 59c6b53 into alstr:master Nov 12, 2024
1 check passed
@alstr
Copy link
Owner

alstr commented Nov 12, 2024

Thanks for all the contributions!

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.

source line is repeated when TODO is on same line as source
2 participants