Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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

@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.

@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.

@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>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants