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.

Sign up for GitHub

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

Jump to bottom

Add GetOrCreate<ListName>Map helper for lists. #971

Merged
merged 1 commit into from
Apr 25, 2024
Merged

Add GetOrCreate<ListName>Map helper for lists. #971

merged 1 commit into from
Apr 25, 2024

Conversation

wenovus
Copy link
Collaborator

@wenovus wenovus commented Apr 25, 2024 •
edited
Loading

This helps avoid nil checks when there is a need to operate on the map field directly.

This is included along the other GetOrCreate helper func.

@wenovus
Add GetOrCreate<ListName>Map helper for lists.
2f02e9f 
This helps avoid nil checks when there is a need to operate on the map
field directly.
@coveralls
Copy link

Coverage Status

coverage: 88.803% (-0.008%) from 88.811%
when pulling 2f02e9f on get-or-create-map
into 9cb751b on master.

ejbrever
ejbrever approved these changes Apr 25, 2024
View reviewed changes
Copy link

@ejbrever ejbrever left a comment

Choose a reason for hiding this comment

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

Thanks!

@wenovus wenovus merged commit c52408e into master Apr 25, 2024
11 checks passed
@wenovus wenovus deleted the get-or-create-map branch April 25, 2024 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ejbrever ejbrever ejbrever approved these changes

@robshakir robshakir Awaiting requested review from robshakir

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wenovus @coveralls @ejbrever