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

Upgrade pure-rust-locales to 0.7.0 (main) #1287

Conversation

jeremija
Copy link
Contributor

@jeremija jeremija commented Sep 14, 2023

This is an update for main branch. PR for main branch is #1288.

Context: chronotope/pure-rust-locales#7

@jeremija jeremija changed the title Upgrade pure-rust-locales to 0.7.0 (main branch for v5) Upgrade pure-rust-locales to 0.7.0 (main) Sep 14, 2023
@jeremija
Copy link
Contributor Author

cc @pitdicker since you reviewed the change in pure-rust-locales.

@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Merging #1287 (897cedb) into main (b64cedc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1287   +/-   ##
=======================================
  Coverage   91.48%   91.48%           
=======================================
  Files          35       35           
  Lines       16860    16860           
=======================================
  Hits        15425    15425           
  Misses       1435     1435           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@djc
Copy link
Contributor

djc commented Sep 14, 2023

@pitdicker I think we can just close this, or would your next merge be easier if we merged this?

(We usually merge regularly from 0.4.x to main so in general there's no need to submit PRs for both branches.)

@jeremija
Copy link
Contributor Author

@djc oh I didn't realize you guys were merging back to main from this branch - in that case I think we can close this (closing now).

@jeremija jeremija closed this Sep 14, 2023
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.

None yet

2 participants