feat : add mapper attribute PreferParameterlessConstructors

[R-abodyak]

add mapper attribute PreferParameterlessConstructors

Controls the priority of constructors used in mapping

When enabled, a parameterless constructor is prioritized over constructors with parameters. When disabled, accessible constructors are ordered in descending order by their parameter count.

Fixes #885

Checklist

• The existing code style is followed
• The commit message follows our guidelines
• Performed a self-review of my code
• Hard-to-understand areas of my code are commented
• The documentation is updated (as applicable)
• Unit tests are added/updated
• Integration tests are added/updated (as applicable, especially if feature/bug depends on roslyn or framework version in use)

[latonz]

Thank you for your first contribution to Mapperly! I added my feedback, feel free to discuss any points or ask questions.

[latonz]

The new public API can be added to the PublicAPI.Unshipped.txt file with the code-fix and then be moved to the PublicAPI.Shipped.txt file right away.

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (cdee78b) 91.18% compared to head (3bf892b) 91.18%.

Files	Patch %	Lines
...apperly/Configuration/MapperConfigurationMerger.cs	75.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1061      +/-   ##
==========================================
- Coverage   91.18%   91.18%   -0.01%     
==========================================
  Files         220      220              
  Lines        7192     7203      +11     
  Branches      913      915       +2     
==========================================
+ Hits         6558     6568      +10     
  Misses        418      418              
- Partials      216      217       +1     

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

[latonz]

A few linter checks are still failing, otherwise this looks good to me 😊

https://github.com/riok/mapperly/actions/runs/7533542937/job/20521692711?pr=1061#step:4:140
https://github.com/riok/mapperly/actions/runs/7533542928/job/20521691733?pr=1061#step:8:11

[R-abodyak]

Hi @latonz , can you check them now ?
btw , is that something I can check locally or not ?

[latonz]

You can run the linters locally. Find the commands in the workflow files:

mapperly/.github/workflows/lint.yml

Lines 18 to 22 in cdee78b

	- run: dotnet tool restore
	- run: dotnet csharpier --check .
	# don't run dotnet format for whitespace formatting as this is done by csharpier
	- run: dotnet format style --verify-no-changes
	- run: dotnet format analyzers --verify-no-changes

mapperly/.github/workflows/docs.yml

Lines 54 to 59 in cdee78b

	- run: npm ci
	working-directory: ./docs
	- run: npm run lint
	working-directory: ./docs
	- run: npm run build
	working-directory: ./docs

Since this is your first PR on this repo I need to approve each pipeline run manually. Further PR pipelines should run automatically.

for the docs you can usually simply run npm run fix in the docs directory.

[github-actions]

🎉 This PR is included in version 3.4.0-next.3 🎉

The release is available on:

• GitHub release
• v3.4.0-next.3

Your semantic-release bot 📦🚀

[github-actions]

🎉 This PR is included in version 3.4.0 🎉

The release is available on:

• GitHub release
• v3.4.0

Your semantic-release bot 📦🚀