Skip to content

Align with the api provided by serde_json #8

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

Merged

Conversation

r-marques
Copy link
Contributor

@r-marques r-marques commented Jan 25, 2019

  • Re-export to_* and from_* methods in the root of the crate
  • Added a rust-toolchain file to set rustup to nightly by default
  • Update README to make it explicit there is a crate published in crates.io

This merge request makes it easier to replace serde_json with serde_json_core when building with no_std.

Example:

#[cfg(not(feature = "std"))]
use serde_json_core as serde_json;
 
let deserialized = serde_json::from_str("...").unwrap();

- Re-export to_* and from_* methods in the root of the crate
- Added a rust-toolchain file to set rustup to nightly by default
- Update README to make it explicit there is a crate published
@japaric
Copy link
Collaborator

japaric commented Feb 5, 2019

Thanks!

bors r+

bors bot added a commit that referenced this pull request Feb 5, 2019
8: Align with the api provided by serde_json r=japaric a=r-marques

- Re-export to_* and from_* methods in the root of the crate
- Added a rust-toolchain file to set rustup to nightly by default
- Update README to make it explicit there is a crate published in crates.io

This merge request makes it easier to replace `serde_json` with `serde_json_core` when building with no_std.

Example:
```rust
#[cfg(not(feature = "std"))]
use serde_json_core as serde_json;
 
let deserialized = serde_json::from_str("...").unwrap();
```

Co-authored-by: Rodolphe Marques <marques.rodolphe@gmail.com>
@bors
Copy link
Contributor

bors bot commented Feb 5, 2019

Build succeeded

@bors bors bot merged commit 48a7e16 into rust-embedded-community:master Feb 5, 2019
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