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

Upgrade @growthbook/growthbook #38

Merged
merged 12 commits into from
Sep 28, 2023
Merged

Upgrade @growthbook/growthbook #38

merged 12 commits into from
Sep 28, 2023

Conversation

bttf
Copy link
Contributor

@bttf bttf commented Sep 13, 2023

Fixes #1664

The JS SDK in devtools was out of date by several versions. Only recently have regressions started coming up (as noted in above linked issue). This PR 1) adds a CI job to catch when @growthbook/growthbook is out-of-date and 2) upgrades said package to latest

Caveats - The opening of a pull request isn't the most ideal time to check whether our SDK is out of date - it's best to check before publishing the extension. However having a CI job check for this the quickest way to have a forcing function somewhere in the lifecycle to ensure we stay up to date. No doubt if this CI check was there prior we would have caught this much earlier.

Changes:

  • Adds a script and a github workflow to run said script upon any pull request that includes changes to src/**/*.ts
  • Upgrades @growthbook/growthbook to 0.28.0

CI Job successfully failing:
Screenshot 2023-09-12 at 8 28 55 PM

@bttf bttf force-pushed the adnan/upgrade-growthbook-js branch from a718ee8 to 016ac9a Compare September 13, 2023 03:09
@bttf bttf marked this pull request as ready for review September 13, 2023 03:09
@jdorn jdorn merged commit 8d19e7b into main Sep 28, 2023
1 check passed
@jdorn jdorn deleted the adnan/upgrade-growthbook-js branch September 28, 2023 19:36
@bttf bttf mentioned this pull request Oct 13, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Given the same user attributes, the JS and React SDKs evaluate to different feature values
2 participants