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

Skip nil check for sync.Once #819

Merged
merged 1 commit into from
Jun 10, 2024
Merged

Skip nil check for sync.Once #819

merged 1 commit into from
Jun 10, 2024

Conversation

charlieegan3
Copy link
Member

When using sync.Once this is not needed.

@charlieegan3 charlieegan3 changed the title Skip nill check Skip nil check for sync.Once Jun 10, 2024
@charlieegan3
Skip nil check
0f1db55 
When using sync.Once this is not needed.

Signed-off-by: Charlie Egan <charlie@styra.com>
@charlieegan3 charlieegan3 merged commit 96246a3 into StyraInc:main Jun 10, 2024
3 checks passed
srenatus pushed a commit to srenatus/regal that referenced this pull request Oct 1, 2024
@charlieegan3 @srenatus
Skip nil check (StyraInc#819)
bed530a 
When using sync.Once this is not needed.

Signed-off-by: Charlie Egan <charlie@styra.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anderseknert anderseknert anderseknert approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@charlieegan3 @anderseknert