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

Feature flags should be modified in-place. #1940

Merged
6 commits merged into from
Nov 27, 2023

Conversation

ghost
Copy link

@ghost ghost commented Nov 15, 2023

Goal

Feature flags position will be in the place when it is added.

Changeset

Feature flags would not be added to the end of the collection when it is modified.

Testing

Exiting unit tests

@ghost ghost requested a review from lemnik November 15, 2023 16:15
@bugsnagbot
Copy link
Collaborator

bugsnagbot commented Nov 15, 2023

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1842.03 1668.63
arm64_v8a 622.85 450.82
armeabi_v7a 561.42 385.29
x86 700.66 524.53
x86_64 671.99 495.86

Generated by 🚫 Danger

@lemnik lemnik requested a review from kstenerud November 16, 2023 10:55
@lemnik lemnik changed the title Feature flags are modified in-place. Feature flags should be modified in-place. Nov 22, 2023
@ghost ghost force-pushed the PLAT-8909/android-feature-flag-behaviour branch from 639a862 to 29c5062 Compare November 27, 2023 11:18
@ghost ghost requested a review from lemnik November 27, 2023 11:25
Copy link
Contributor

@lemnik lemnik left a comment

Choose a reason for hiding this comment

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

LGTM

SmartbearYing added 2 commits November 27, 2023 11:48
* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack
@ghost ghost merged commit ac426b0 into next Nov 27, 2023
@ghost ghost deleted the PLAT-8909/android-feature-flag-behaviour branch November 27, 2023 12:27
ghost pushed a commit that referenced this pull request Nov 27, 2023
* fix(feature flags) refactor feature flags are modified in-place.

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack
ghost pushed a commit that referenced this pull request Nov 28, 2023
* fix(payload size) added error call back to make the payload size more consistent

* Convert file store and subclasses to kotlin (#1929)

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack

* Feature flags should be modified in-place. (#1940)

* fix(feature flags) refactor feature flags are modified in-place.

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack

* fix(payload size) adjusted end to end test
ghost pushed a commit that referenced this pull request Dec 5, 2023
* fix(payload size) added error call back to make the payload size more consistent

* Convert file store and subclasses to kotlin (#1929)

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack

* Feature flags should be modified in-place. (#1940)

* fix(feature flags) refactor feature flags are modified in-place.

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack

* fix(payload size) adjusted end to end test
@ghost ghost mentioned this pull request Dec 5, 2023
ghost pushed a commit that referenced this pull request Dec 5, 2023
* Update workflow to use auth provided by bearer token

* docs(upgrade guide): expanded UG content and a few corrections to changelog

* docs(upgrade guide): emphasis stability score calculation

* docs(upgrade guide): add persist user change

* Convert file store and subclasses to kotlin (#1929)

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* ANR Scenario session should be allowed to terminate properly (#1948)

* fix(anr scenario ) add more time to wait for appium to figure out which session is terminated

* v6.0.0

* corrections to fix v6 publishing

* add the exitinfo module to the publishing step

* fix(foreground): ensure inForeground is set correctly when the app is backgrounded, and reintroduce the test scenario

* chore(changelog): updated changelog to reflect new release date for v6.0.0

* Update workflow to use auth provided by bearer token

* docs(upgrade guide): expanded UG content and a few corrections to changelog

* docs(upgrade guide): emphasis stability score calculation

* docs(upgrade guide): add persist user change

---------

Co-authored-by: jason <lemnik@users.noreply.github.com>
Co-authored-by: Alex Moinet <alex@bugsnag.com>
Co-authored-by: Tom Longridge <tom.longridge@smartbear.com>

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack

* Feature flags should be modified in-place. (#1940)

* fix(feature flags) refactor feature flags are modified in-place.

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack

* Consistent payload (#1949)

* fix(payload size) added error call back to make the payload size more consistent

* Convert file store and subclasses to kotlin (#1929)

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* fix(code convert) add FileStore, sessionStore and EventStore Kotlin classes

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack

* Feature flags should be modified in-place. (#1940)

* fix(feature flags) refactor feature flags are modified in-place.

* Time limitation for thread collection (#1935)

* fix(threadStack timeout) add time limitation for thread stack

* fix(threadStack timeout) add time limitation for thread stack

* fix(payload size) adjusted end to end test

* fix(ndk): replace `static_json_data` with an atomic to ensure that it cannot be double-freed

* release/v6.1.0

---------

Co-authored-by: Jason <lemnik@users.noreply.github.com>
Co-authored-by: Alex Moinet <alex@bugsnag.com>
Co-authored-by: Tom Longridge <tom.longridge@smartbear.com>
Co-authored-by: Tom Longridge <tom@bugsnag.com>
This pull request was closed.
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.

3 participants