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

derive: use hygenic path for Result in output #19

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

spikespaz
Copy link

This stops conflicts with too many generics when type aliases of Result are in scope.

@spikespaz spikespaz force-pushed the derive-hygienic-paths branch 2 times, most recently from 40bd894 to ebce72a Compare December 30, 2024 02:22
@spikespaz spikespaz changed the title derive: scalar: emit_enum: use hygenic path for Result derive: use hygenic path for Result Dec 30, 2024
@spikespaz spikespaz changed the title derive: use hygenic path for Result derive: use hygenic path for Result in output Dec 30, 2024
This stops conflicts with too many generics when type
aliases of `Result` are in scope.
@spikespaz spikespaz force-pushed the derive-hygienic-paths branch from ebce72a to 72323e4 Compare December 30, 2024 02:27
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.26%. Comparing base (163f329) to head (72323e4).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #19   +/-   ##
=======================================
  Coverage   76.26%   76.26%           
=======================================
  Files          15       15           
  Lines        4272     4272           
  Branches     4272     4272           
=======================================
  Hits         3258     3258           
  Misses        872      872           
  Partials      142      142           

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

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