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

Go tutorial 🐿️ #8904

Merged
merged 10 commits into from
Dec 13, 2024
Merged

Go tutorial 🐿️ #8904

merged 10 commits into from
Dec 13, 2024

Conversation

alvinometric
Copy link
Contributor

And examples πŸ˜„

Copy link

vercel bot commented Dec 2, 2024

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
unleash-docs βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Dec 13, 2024 8:36pm
unleash-monorepo-frontend βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Dec 13, 2024 8:36pm

Copy link
Contributor

github-actions bot commented Dec 2, 2024

Dependency Review

βœ… No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link
Collaborator

@melindafekete melindafekete left a comment

Choose a reason for hiding this comment

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

Nice! πŸš€ Left some suggestions to align with the style guide. Please use up-to-date screenshots πŸ™ (also with better aspect ratio if possible). Let me know if you need help


## 5. Add Unleash to your Go app

Now, let's connect our project to Unleash so that you can toggle that feature flag at runtime. If you wanted to, you could also do a gradual rollout, use it for a/b testing, etc.
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if linking to the gradual rollout process or a/b testing would be good here.

Co-authored-by: Melinda Fekete <[email protected]>
Co-authored-by: Nnenna Ndukwe <[email protected]>
Copy link
Contributor

@FredrikOseberg FredrikOseberg left a comment

Choose a reason for hiding this comment

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

Apologies if I sound overly critical, I'm probably just missing some context. Some general feedback:

  • Go code looks idiomatic
  • Section using SDK looks good

I was having trouble understanding the first section of this PR where we provide examples on how to integrate with our API. There are a couple of reasons why:

  • When I read feature flagging guide, unless it's made very clear, I assume that the guide is on consumption of feature flags.
  • Language specific examples may be useful if you are writing a golang application that for some reason needs to manipulate the Admin API, but is this common? This is primarily something you would do via the Unleash UI. If I was going to automate this I would probably write bash scripts. It's typically not something you do from the application that consumes Unleash. I wonder if it just causing confusion?

@alvinometric
Copy link
Contributor Author

Hey, thanks for the review @FredrikOseberg I said this in another comment but the Examples part is something we've been doing since the beginning and that is mostly for reference purposes.

It's so that people have easy blocks of code to copy-paste for various actions in their favourite languages. We've done this for every language we support so far.

That doesn't mean we have to continue doing it πŸ˜… If you think these examples are counterproductive then we can talk about removing them

I removed all the problematic sections that you pointed out πŸ™

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants