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.

Sign up for GitHub

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

Jump to bottom

std json #159

Merged
merged 9 commits into from
Jul 26, 2023
Merged

std json #159

merged 9 commits into from
Jul 26, 2023

Conversation

jordens
Copy link
Member

@jordens jordens commented Jul 26, 2023 •
edited
Loading

@jordens
make no heapless needed for serde_json_core
00d6f68
@jordens
remove heapless dependency from json feature
9872480
@jordens
rename features, add std json
5bfc4d3 
* rename json -> json-core
* rename mqtt -> mqtt-client (back)
* add json feature and serde-json with slash style serde (requires std)
* remove heapless dependency from json-core
@jordens
Merge remote-tracking branch 'origin/main' into deps
817622e 
* origin/main:
@jordens
use fixed minimq
25bdf4f
@jordens
test default features
7836eea
@jordens
Copy link
Member Author

jordens commented Jul 26, 2023

The downside to the Serde<Style> thing is that once you have multiple enabled at the same time (e.g. SerDe<SCPI> and SerDe<JsonCoreSlash> for different simultaneous interfaces), you have to explicitly disambiguate each and every call to the SerDe trait.

@ryan-summers ryan-summers merged commit 4f0a6aa into main Jul 26, 2023
14 checks passed
@ryan-summers ryan-summers deleted the std-json branch July 26, 2023 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ryan-summers ryan-summers ryan-summers approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jordens @ryan-summers