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

fix: remove interface from attribute targets on the mapper attribute #686

Merged
merged 1 commit into from Aug 28, 2023

Conversation

latonz
Copy link
Contributor

@latonz latonz commented Aug 28, 2023

No description provided.

@latonz latonz self-assigned this Aug 28, 2023
@codecov
Copy link

codecov bot commented Aug 28, 2023

Codecov Report

Merging #686 (bb8c189) into main (f3d78b4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #686   +/-   ##
=======================================
  Coverage   90.98%   90.98%           
=======================================
  Files         172      172           
  Lines        6132     6132           
  Branches      781      781           
=======================================
  Hits         5579     5579           
  Misses        377      377           
  Partials      176      176           
Files Changed Coverage Δ
src/Riok.Mapperly.Abstractions/MapperAttribute.cs 100.00% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@latonz latonz enabled auto-merge (squash) August 28, 2023 10:26
@latonz latonz requested review from CommonGuy and ni507 August 28, 2023 10:26
Mapperly does not work with interfaces since v2.0.0
@latonz latonz merged commit b7e6e20 into riok:main Aug 28, 2023
17 checks passed
@latonz latonz deleted the fix/mapper-attribute-usage branch August 28, 2023 12:06
@github-actions
Copy link

🎉 This PR is included in version 3.2.0-next.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This PR is included in version 3.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants