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.

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

perf(core): lazy load usage annotations allocation #2701

Merged

Conversation

Codelax
Copy link
Member

@Codelax Codelax commented Dec 27, 2022

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #0000

Release note for CHANGELOG:


@Codelax Codelax added the core label Dec 27, 2022
@Codelax Codelax self-assigned this Dec 27, 2022
@codecov-commenter
Copy link

codecov-commenter commented Dec 27, 2022

Codecov Report

Merging #2701 (d018378) into master (d999184) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2701      +/-   ##
==========================================
- Coverage   75.56%   75.55%   -0.02%     
==========================================
  Files         134      134              
  Lines       25029    25031       +2     
==========================================
- Hits        18914    18913       -1     
- Misses       5475     5477       +2     
- Partials      640      641       +1     
Impacted Files Coverage Δ
internal/core/cobra_builder.go 87.09% <100.00%> (+0.28%) ⬆️
internal/core/command.go 50.73% <0.00%> (-2.21%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@remyleone remyleone merged commit efb08dd into scaleway:master Dec 27, 2022
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.

3 participants