Skip to content

Fix structaccess.Get to handle embedded structs and empty slices/maps - #3657

Merged
denik merged 9 commits into
mainfrom
denik/structaccess-get-fsf
Sep 25, 2025
Merged

Fix structaccess.Get to handle embedded structs and empty slices/maps#3657
denik merged 9 commits into
mainfrom
denik/structaccess-get-fsf

Conversation

@denik

@denikdenik commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Why

Fixing bugs, this is used in direct deployment for resolving resource references.

Tests are adapted from #3650 it's the same issue in different libraries.

Tests

New unit tests.

@denik
denik changed the base branch from main to denik/update-get-resource-configSeptember 25, 2025 08:53
@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Sep 25, 2025

Copy link
Copy Markdown
Collaborator

Run: 18007397265

Env✅​pass🙈​skip
✅​aws linux312530
✅​aws windows313529
✅​aws-ucws linux425427
✅​aws-ucws windows426426
✅​azure linux312529
✅​azure windows313528
✅​azure-ucws linux425426
✅​azure-ucws windows426425
✅​gcp linux311531
✅​gcp windows312530

@denik
denikforce-pushed the denik/structaccess-get-fsf branch from 9297462 to c14f9e6CompareSeptember 25, 2025 12:14
@denik
denik changed the base branch from denik/update-get-resource-config to mainSeptember 25, 2025 12:18
@denik
denik enabled auto-merge September 25, 2025 12:19
@denik
denikforce-pushed the denik/structaccess-get-fsf branch from 5280d2c to 0f21130CompareSeptember 25, 2025 12:25
@denik
denik disabled auto-merge September 25, 2025 12:31
@denik
denik merged commit 3ba8c2e into mainSep 25, 2025
12 of 13 checks passed
@denik
denik deleted the denik/structaccess-get-fsf branch September 25, 2025 12:31
denik added a commit that referenced this pull request May 20, 2026
…#3657)
## Why
Fixing bugs, this is used in direct deployment for resolving resource
references.
Tests are adapted from #3650 it's
the same issue in different libraries.
## Tests
New unit tests.
Sign up for freeto 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.

3 participants

@denik@eng-dev-ecosystem-bot@andrewnester