Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: update comments for named provider related function #213

Merged
merged 3 commits into from
Sep 6, 2023

Conversation

yongruilin
Copy link
Contributor

This PR

Update comments for named provider related functions.

Related Issues

N/A

Notes

Only updates comments.

Follow-up Tasks

N/A

How to test

N/A

@yongruilin yongruilin requested a review from a team as a code owner September 5, 2023 06:12
@yongruilin
Copy link
Contributor Author

@beeme1mr, @Kavindu-Dodan PTAL, thanks!

@codecov
Copy link

codecov bot commented Sep 5, 2023

Codecov Report

Merging #213 (a4c9d2e) into main (e79382a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #213   +/-   ##
=======================================
  Coverage   78.63%   78.63%           
=======================================
  Files          10       10           
  Lines        1203     1203           
=======================================
  Hits          946      946           
  Misses        230      230           
  Partials       27       27           
Files Changed Coverage Δ
pkg/openfeature/api.go 92.41% <ø> (ø)
pkg/openfeature/openfeature.go 97.67% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@beeme1mr
Copy link
Member

beeme1mr commented Sep 5, 2023

Thanks @yongruilin! Could you please sign off your commits? It's a requirement from the CNCF before we can merge.

Signed-off-by: yongruilin <yongrlin@outlook.com>
@beeme1mr
Copy link
Member

beeme1mr commented Sep 5, 2023

It looks like there's an unrelated build error. I'll look into it tomorrow.

Fyi @toddbaert @thisthat @Kavindu-Dodan

@thisthat
Copy link
Member

thisthat commented Sep 6, 2023

It looks like there's an unrelated build error. I'll look into it tomorrow.

Fyi @toddbaert @thisthat @Kavindu-Dodan

I guess it is the same problem identified by @Kavindu-Dodan in the contrib repo: https://github.com/open-feature/go-sdk-contrib/pull/317/files#r1306174491

@toddbaert
Copy link
Member

I've opened #216 to fix the ci issue.

@toddbaert toddbaert closed this Sep 6, 2023
@toddbaert toddbaert reopened this Sep 6, 2023
@beeme1mr beeme1mr merged commit 2e670b2 into open-feature:main Sep 6, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants