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 implementation of block commets #168

Merged
merged 3 commits into from
Dec 15, 2024
Merged

New implementation of block commets #168

merged 3 commits into from
Dec 15, 2024

Conversation

ppolesiuk
Copy link
Member

The behavior of block comments is changed, as proposed in the documentation of Fram.

We can wait with merging this PR, until we agree on the approach to block comments. But I hope, this PR shows that the solution proposed in #166 has a simple implementation.

Fixes #166

The behavior of block comments is changed, as proposed in the
documentation of Fram.
@ppolesiuk ppolesiuk marked this pull request as ready for review December 14, 2024 22:42
@ppolesiuk ppolesiuk requested a review from forell December 15, 2024 06:24
Copy link
Member

@forell forell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, and I appreciate how even the implementation is simpler than with the previous approach (which was missing various edge cases anyway). Are we still waiting for feedback before merging?

@ppolesiuk
Copy link
Member Author

I think, I was loud enough on discord, and we've got positive feedback on Friday. But I would still wait, until you accept (or reject?) another pull-request to the documentation. If it will be accepted, I can change the implementation.

@forell forell merged commit 836d7bf into master Dec 15, 2024
2 checks passed
@forell forell deleted the 166-comments branch December 15, 2024 20:04
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.

Awkward interaction of doc comments and single-line comments
2 participants