Skip to content

fix!: Iterate on the API #198

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 9 commits into from
Apr 16, 2025
Merged

fix!: Iterate on the API #198

merged 9 commits into from
Apr 16, 2025

Conversation

epage
Copy link
Contributor

@epage epage commented Apr 16, 2025

This is basically review feedback on top of #195

Some things were deferred out, like #196, #197

Fixes #175

BREAKING CHANGE: Renamed Renderer::line_no to Renderer::line_num

@epage epage changed the title fix: Iterate on the API fix!: Iterate on the API Apr 16, 2025
Copy link
Member

@Muscraft Muscraft left a comment

Choose a reason for hiding this comment

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

These changes look good to me!

@Muscraft Muscraft merged commit a78e165 into rust-lang:master Apr 16, 2025
15 checks passed
@epage epage deleted the fixes branch April 17, 2025 01:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Renderer::line_no is a confusing name
2 participants