Skip to content
This repository was archived by the owner on Dec 17, 2025. It is now read-only.

Add const support to inherent impl and attr support generally#53

Merged
VictorKoenders merged 2 commits intobincode-org:trunkfrom
Daniel-Aaron-Bloom:dbloom/const-attr
May 14, 2023
Merged

Add const support to inherent impl and attr support generally#53
VictorKoenders merged 2 commits intobincode-org:trunkfrom
Daniel-Aaron-Bloom:dbloom/const-attr

Conversation

@Daniel-Aaron-Bloom
Copy link
Contributor

I think this resolves #33, but I have not tested any of this code.

@codecov
Copy link

codecov bot commented Apr 17, 2023

Codecov Report

Patch coverage: 7.05% and project coverage change: -1.62 ⚠️

Comparison is base (789c14a) 50.07% compared to head (d510e77) 48.46%.

Additional details and impacted files
@@            Coverage Diff             @@
##            trunk      #53      +/-   ##
==========================================
- Coverage   50.07%   48.46%   -1.62%     
==========================================
  Files          19       19              
  Lines        1945     2018      +73     
==========================================
+ Hits          974      978       +4     
- Misses        971     1040      +69     
Impacted Files Coverage Δ
src/generate/generate_item.rs 0.00% <0.00%> (ø)
src/generate/impl.rs 0.00% <0.00%> (ø)
src/generate/impl_for.rs 41.00% <22.22%> (-2.03%) ⬇️

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Daniel-Aaron-Bloom
Copy link
Contributor Author

@VictorKoenders is this change something you're interested in?

@VictorKoenders
Copy link
Contributor

Yes I think this looks good.

I think currently virtue does not have a good way to test changes like this, so I need to think about how we're going to test new features in PRs like this. I'll think about this testing framework for a while and get back to this PR when I figure something out.

So it'll be a bit before I merge this, sorry!

@VictorKoenders
Copy link
Contributor

No good solution has popped up yet and I don't want this to be blocked for too long, so I'll merge this. Thanks!

@VictorKoenders VictorKoenders merged commit ff02e57 into bincode-org:trunk May 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to put attribute #[async_trait] before impl block?

2 participants