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

Bump to 5.1.0 #487

Merged
merged 1 commit into from
Jul 31, 2023
Merged

Bump to 5.1.0 #487

merged 1 commit into from
Jul 31, 2023

Conversation

Enselic
Copy link
Collaborator

@Enselic Enselic commented Jul 30, 2023

Draft of release notes:

Improvements

  • Fix how syntect::util::split_at() handles multi-byte characters
  • Allow case insensitive search for .tmtheme paths
  • impl PartialEq for Theme

Dependencies

  • Upgrade fancy-regex to 0.11
  • Upgrade regex-syntax to 0.7
  • Replace lazy_static with once_cell

Other

  • Change MSRV policy to "last three stable versions"
  • Make Debug impl of syntect::highlighting::Color less verbose

(I have confirmed that this version of syntect passes all bat regression tests: sharkdp/bat#2632)

@Enselic
Copy link
Collaborator Author

Enselic commented Jul 30, 2023

Note: Strictly speaking this should be 6.0.0 because we remove public API, but we have previously decided to take the risk. See #438 (review) and the rest of the discussion for details.

Copy link
Owner

@trishume trishume left a comment

Choose a reason for hiding this comment

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

Thanks for putting this together!

@trishume
Copy link
Owner

How about you try doing the release yourself and let me know if you run into any permissions snags.

@Enselic
Copy link
Collaborator Author

Enselic commented Jul 31, 2023

Sounds good! Will try.

@Enselic Enselic merged commit b7531a8 into trishume:master Jul 31, 2023
3 checks passed
@Enselic Enselic deleted the bump branch July 31, 2023 16:12
@Enselic
Copy link
Collaborator Author

Enselic commented Jul 31, 2023

Went smoothly! (knock on wood)

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