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

feat: support parameterized constructors #51

Merged
merged 1 commit into from
Mar 15, 2022
Merged

Conversation

latonz
Copy link
Contributor

@latonz latonz commented Mar 14, 2022

No description provided.

@latonz latonz self-assigned this Mar 14, 2022
@latonz latonz linked an issue Mar 14, 2022 that may be closed by this pull request
@latonz latonz force-pushed the feat/support-records branch 2 times, most recently from 9f8ef24 to 0c493b5 Compare March 14, 2022 07:12
@codecov-commenter
Copy link

codecov-commenter commented Mar 14, 2022

Codecov Report

Merging #51 (a94351a) into main (077fa72) will decrease coverage by 0.79%.
The diff coverage is 85.39%.

@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
- Coverage   89.83%   89.04%   -0.80%     
==========================================
  Files          66       71       +5     
  Lines        1968     2199     +231     
  Branches      261      286      +25     
==========================================
+ Hits         1768     1958     +190     
- Misses        138      175      +37     
- Partials       62       66       +4     
Impacted Files Coverage Δ
...gs/PropertyMappings/ConstructorParameterMapping.cs 57.50% <57.50%> (ø)
...ings/PropertyMappings/PropertyAssignmentMapping.cs 62.79% <62.79%> (ø)
...s/Mappings/PropertyMappings/NullPropertyMapping.cs 66.66% <66.66%> (ø)
...yMappings/PropertyNullDelegateAssignmentMapping.cs 70.58% <75.00%> (ø)
...Builder/NewInstanceObjectPropertyMappingBuilder.cs 85.88% <85.88%> (ø)
...criptors/Mappings/PropertyMappings/PropertyPath.cs 87.12% <89.36%> (+0.68%) ⬆️
src/Riok.Mapperly/Descriptors/DescriptorBuilder.cs 97.91% <100.00%> (+0.02%) ⬆️
...scriptors/MappingBuilder/NullableMappingBuilder.cs 95.83% <100.00%> (-1.54%) ⬇️
...ors/MappingBuilder/ObjectPropertyMappingBuilder.cs 100.00% <100.00%> (+0.53%) ⬆️
...pingBuilder/ObjectPropertyMappingBuilderContext.cs 100.00% <100.00%> (ø)
... and 14 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@latonz latonz force-pushed the feat/support-records branch 3 times, most recently from 0b38cbb to b3dce9f Compare March 14, 2022 10:49
@latonz latonz marked this pull request as ready for review March 14, 2022 10:54
@latonz latonz requested a review from CommonGuy March 14, 2022 10:54
@CommonGuy CommonGuy merged commit b99c96a into main Mar 15, 2022
@CommonGuy CommonGuy deleted the feat/support-records branch March 15, 2022 06:56
@github-actions
Copy link

🎉 This PR is included in version 2.2.0-next.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This PR is included in version 2.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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support parameterised constructors
3 participants