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

feat: automatically generated instanceId #637

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

nunogois
Copy link
Member

@nunogois nunogois commented Jul 9, 2024

https://linear.app/unleash/issue/2-2397/make-instance-id-autogenned-node

This makes instanceId an automatically generated property, instead of being set by the user.

Similar to Unleash/unleash-client-ruby#179
Similar to Unleash/unleash-client-dotnet#226

@nunogois nunogois requested a review from sighphyre July 9, 2024 14:51
@coveralls
Copy link

Coverage Status

coverage: 90.834% (+0.06%) from 90.772%
when pulling f474dc8 on feat-auto-generated-instanceid
into 0198e0a on main.

@alvinometric
Copy link
Contributor

Not sure it needs a major version because the code will still work if you pass an object with an instanceId

Copy link
Member

@sighphyre sighphyre left a comment

Choose a reason for hiding this comment

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

Ye!

@sighphyre
Copy link
Member

Not sure it needs a major version because the code will still work if you pass an object with an instanceId

It matters for TypeScript, sadly (actually strongly mixed feelings?). I thought the same thing so I went and checked and yes the compiler does complain

@nunogois nunogois merged commit 279cd2e into main Jul 10, 2024
5 checks passed
@nunogois nunogois deleted the feat-auto-generated-instanceid branch July 10, 2024 08:10
@alvinometric
Copy link
Contributor

:old-man-yells-at-typescript:

nunogois added a commit that referenced this pull request Jul 10, 2024
This reverts commit 279cd2e.
nunogois added a commit that referenced this pull request Jul 10, 2024
This reverts commit 279cd2e.
nunogois added a commit that referenced this pull request Jul 10, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
This reverts commit a3f3326.
nunogois added a commit that referenced this pull request Jul 10, 2024
This reverts commit a3f3326.
@naderghanbari
Copy link

What about Unleash-API-compatible providers like Gitlab Feature Flags, that do require an instanceId? Does this mean the sdks/clients will be incompatible with these providers after this change?

nunogois added a commit that referenced this pull request Jul 15, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…) (#641)"

This reverts commit 5b9da4e.
nunogois added a commit that referenced this pull request Jul 15, 2024
This reverts commit 5b9da4e.
renovate bot referenced this pull request in Unleash/unleash Sep 9, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[unleash-client](https://redirect.github.com/Unleash/unleash-client-node)
| [`5.5.5` ->
`5.6.1`](https://renovatebot.com/diffs/npm/unleash-client/5.5.5/5.6.1) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/unleash-client/5.6.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/unleash-client/5.6.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/unleash-client/5.5.5/5.6.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/unleash-client/5.5.5/5.6.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>Unleash/unleash-client-node (unleash-client)</summary>

###
[`v5.6.1`](https://redirect.github.com/Unleash/unleash-client-node/releases/tag/v5.6.1)

[Compare
Source](https://redirect.github.com/Unleash/unleash-client-node/compare/v5.6.0...v5.6.1)

#### What's Changed

- Revert "feat: automatically generated instanceId
([#&#8203;637](https://redirect.github.com/Unleash/unleash-client-node/issues/637))"
by [@&#8203;nunogois](https://redirect.github.com/nunogois) in
[https://github.com/Unleash/unleash-client-node/pull/640](https://redirect.github.com/Unleash/unleash-client-node/pull/640)

**Full Changelog**:
Unleash/unleash-client-node@v5.6.0...v5.6.1

###
[`v5.6.0`](https://redirect.github.com/Unleash/unleash-client-node/releases/tag/v5.6.0)

[Compare
Source](https://redirect.github.com/Unleash/unleash-client-node/compare/v5.5.5...v5.6.0)

#### What's Changed

- chore(deps): update dependency typescript to v5.5.2 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[https://github.com/Unleash/unleash-client-node/pull/631](https://redirect.github.com/Unleash/unleash-client-node/pull/631)
- chore(deps): update unleash/.github action to v1.2.0 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[https://github.com/Unleash/unleash-client-node/pull/632](https://redirect.github.com/Unleash/unleash-client-node/pull/632)
- feat: automatically generated instanceId by
[@&#8203;nunogois](https://redirect.github.com/nunogois) in
[https://github.com/Unleash/unleash-client-node/pull/637](https://redirect.github.com/Unleash/unleash-client-node/pull/637)
- feat: http and https proxy support by
[@&#8203;kwasniew](https://redirect.github.com/kwasniew) in
[https://github.com/Unleash/unleash-client-node/pull/638](https://redirect.github.com/Unleash/unleash-client-node/pull/638)

**Full Changelog**:
Unleash/unleash-client-node@v5.5.5...v5.6.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 7pm every weekday,before 5am
every weekday" in timezone Europe/Madrid, Automerge - At any time (no
schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/Unleash/unleash).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41OS4yIiwidXBkYXRlZEluVmVyIjoiMzguNTkuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

None yet

5 participants