Skip to content

Conversation

jonasbb
Copy link
Owner

@jonasbb jonasbb commented Apr 21, 2021

Properly name all types and traits in the derive macros such that they do not conflict with the environment where they are used.

bors r+

bors bot added a commit that referenced this pull request Apr 21, 2021
305: Fix proc-macros were relying on the prelude to be available r=jonasbb a=jonasbb

Properly name all types and traits in the derive macros such that they do not conflict with the environment where they are used.

bors r+

Co-authored-by: Jonas Bushart <jonas@bushart.org>
@bors
Copy link
Contributor

bors bot commented Apr 21, 2021

Build failed:

@codecov
Copy link

codecov bot commented Apr 21, 2021

Codecov Report

Merging #305 (e1ac385) into master (3e2a780) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #305   +/-   ##
=======================================
  Coverage   78.81%   78.81%           
=======================================
  Files          41       41           
  Lines        2483     2483           
=======================================
  Hits         1957     1957           
  Misses        526      526           
Impacted Files Coverage Δ
serde_with_macros/src/lib.rs 0.00% <ø> (ø)
serde_with_test/tests/derive_display_fromstr.rs 0.00% <ø> (ø)
serde_with_test/tests/flattened_maybe.rs 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e2a780...e1ac385. Read the comment docs.

@jonasbb
Copy link
Owner Author

jonasbb commented Apr 21, 2021

bors r+

@bors
Copy link
Contributor

bors bot commented Apr 21, 2021

Build succeeded:

@bors bors bot merged commit f72726d into master Apr 21, 2021
@bors bors bot deleted the macro-hygiene branch April 21, 2021 22:07
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.

1 participant