Skip to content

fix: retract v11.0.1, gate init nil pointers#318

Merged
caarlos0 merged 5 commits intomainfrom
init
Jun 19, 2024
Merged

fix: retract v11.0.1, gate init nil pointers#318
caarlos0 merged 5 commits intomainfrom
init

Conversation

@caarlos0
Copy link
Copy Markdown
Owner

@caarlos0 caarlos0 commented Jun 19, 2024

as it would automatically initialize nil pointers.

this retracts that version, and gate this new feature behind an init tag option.

closes #317
closes #310
refs #306
refs #234

as it would automatically initialize nil pointers.

this retracts that version, and gate this new feature behind an `init`
tag option.

closes #317
refs  #306
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (de7a9cc) to head (205b4b0).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #318   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          427       429    +2     
=========================================
+ Hits           427       429    +2     

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

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@pull-request-size pull-request-size bot added size/L and removed size/M labels Jun 19, 2024
caarlos0 added 2 commits June 19, 2024 10:33
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 merged commit 432567c into main Jun 19, 2024
@caarlos0 caarlos0 deleted the init branch June 19, 2024 13:37
BorzdeG pushed a commit to BorzdeG/env that referenced this pull request Aug 24, 2024
* fix: retract v11.0.1, gate init nil pointers

as it would automatically initialize nil pointers.

this retracts that version, and gate this new feature behind an `init`
tag option.

closes caarlos0#317
refs  caarlos0#306

* test: make sure caarlos0#310 is covered too

closes caarlos0#310

* test: add test for multiple tag options

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

* fix: typo

* perf: cheap first

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>

---------

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
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.

URL Pointers are no longer left nil Don't touch struct fields without env tag

1 participant