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: mapping slice of complex struct #312

Merged
merged 8 commits into from
Jul 19, 2024

Conversation

Rancbar
Copy link
Contributor

@Rancbar Rancbar commented May 28, 2024

feat: mapping slice of complex struct. #298

Unverified

This user has not yet uploaded their public signing key.
@dongjiang1989
Copy link

LGTM

Copy link

codecov bot commented Jun 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (0de9383) to head (ece55fb).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #312   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          429       498   +69     
=========================================
+ Hits           429       498   +69     

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

Hamid Reza Ranjbar added 4 commits June 2, 2024 21:18
@dongjiang1989
Copy link

/cc @caarlos0 PTAL, thanks

@Rancbar
Copy link
Contributor Author

Rancbar commented Jun 30, 2024

Hi @caarlos0, could you please review this PR when you have a moment? I am happy to make any changes if needed. Thank you!

Copy link
Owner

@caarlos0 caarlos0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey, it looks to me!

sorry for the delay reviewing.

I think the only thing missing is something in the readme?
PS: I know the readme is already convoluted, I need to clean it up at some point.

@Rancbar
Copy link
Contributor Author

Rancbar commented Jul 2, 2024

Hey, Happy to hear that
Sure, I will update the readme in the coming couple of days.

Hamid Reza Ranjbar added 2 commits July 15, 2024 23:16
@Rancbar
Copy link
Contributor Author

Rancbar commented Jul 15, 2024

Dear @caarlos0, I added a sample to the readme. Please check it up when you can

@caarlos0 caarlos0 merged commit 68793c0 into caarlos0:main Jul 19, 2024
15 checks passed
@caarlos0
Copy link
Owner

many thanks, this looks great!

@caarlos0
Copy link
Owner

caarlos0 commented Aug 6, 2024

see #320 #321

BorzdeG pushed a commit to BorzdeG/env that referenced this pull request Aug 24, 2024
* feat: mapping slice of complex struct caarlos0#298

* feat: support predefined values and pre initialized structs

* refactor: some improvement

* refactor: some improvement

* test: support normal features for nested fields

* test: trying to fix `gofumpt` lint issues

* chore: add sample for complex struct in readme

---------

Co-authored-by: Hamid Reza Ranjbar <hamidreza.ranjbar@snapp.cab>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants