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 ron to v0.8.1 #472

Merged
merged 30 commits into from
Aug 17, 2023
Merged

Bump ron to v0.8.1 #472

merged 30 commits into from
Aug 17, 2023

Conversation

juntyr
Copy link
Member

@juntyr juntyr commented Aug 15, 2023

Fixes #469

  • I've included my change in CHANGELOG.md

juntyr and others added 29 commits August 15, 2023 23:10
* Format expected bytes error with base64 as in ron

* Reformat imports to be consistent
* Fixed raw struct name identifier parsing

* Added more tests, raw identifier suggestion error, and invalid identifier error formatting

* Even more empty identifier tests
…on-rs#415)

* Error instead of panic when deserializing non-identifier field name

* Fix code coverage for tests

* Added CHANGELOG entry
* Explicit recursion limit during deserializing

* Explicit recursion limit during serializing

* Added CHANGELOG entry
…on-rs#441)

* depend on serde_derive separately for more compilation parallelism

* add changelog entry
* Upgrade to bitflags 2.0

* Update the fuzzer Cargo.lock
…rde data + fix `&[u8]` de (ron-rs#462)

* Add a fuzzer test that serialises and deserialises arbitrary typed serde data + fix &[u8] de

* Fix the byte slice serialising test
* trim trailing whitespace in README 

* fix 'wasd' key map in README example, very important
@juntyr juntyr self-assigned this Aug 15, 2023
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 82.07% and project coverage change: -2.43% ⚠️

Comparison is base (4089034) 86.55% compared to head (b4f1cdf) 84.13%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##             v0.8     #472      +/-   ##
==========================================
- Coverage   86.55%   84.13%   -2.43%     
==========================================
  Files          49       56       +7     
  Lines        5803     6775     +972     
==========================================
+ Hits         5023     5700     +677     
- Misses        780     1075     +295     
Files Changed Coverage Δ
src/de/value.rs 87.40% <ø> (ø)
src/lib.rs 0.00% <ø> (-100.00%) ⬇️
src/ser/tests.rs 100.00% <ø> (ø)
tests/147_empty_sets_serialisation.rs 97.67% <ø> (-2.33%) ⬇️
tests/203_error_positions.rs 98.75% <ø> (+12.72%) ⬆️
tests/250_variant_newtypes.rs 99.71% <ø> (-0.29%) ⬇️
tests/307_stack_overflow.rs 97.91% <ø> (ø)
tests/depth_limit.rs 96.42% <ø> (-3.58%) ⬇️
tests/escape.rs 98.43% <0.00%> (-1.57%) ⬇️
tests/extensions.rs 96.55% <ø> (-3.45%) ⬇️
... and 25 more

... and 17 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@juntyr
Copy link
Member Author

juntyr commented Aug 16, 2023

@torkleyy @kvark I've gone through all changes to the main branch since v0.8.0 and cherry-picked all bug fixes and non-API-breaking changes. In particular, the following commits are excluded:

but will be added in the v0.9 release.

@juntyr juntyr linked an issue Aug 16, 2023 that may be closed by this pull request
@juntyr
Copy link
Member Author

juntyr commented Aug 16, 2023

@extrawurst I hope I can get this published as soon as I get the necessary rights. The bump includes all bug fixes and dependency upgrades but excludes work on making ron work better with serde attributes and values, which is still in progress. Does this look good to you?

@extrawurst
Copy link

Lgtm

@juntyr juntyr merged commit eafa2b6 into ron-rs:v0.8 Aug 17, 2023
6 checks passed
@juntyr juntyr deleted the v0.8.1 branch August 17, 2023 05:09
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.

new release bumping bitflags dependency
6 participants