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.

Sign up for GitHub

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

Jump to bottom

fix: race conditions #187

Closed
wants to merge 41 commits into from
Closed

fix: race conditions #187

wants to merge 41 commits into from

Conversation

1abhishekpandey
Copy link
Contributor

@1abhishekpandey 1abhishekpandey commented Jul 14, 2022 •
edited by pallabmaiti
Loading

Description

Fixed couple of race conditions & bugs.

@pallabmaiti pallabmaiti changed the base branch from crashfix/fetchconfig to develop July 19, 2022 04:59
@pallabmaiti pallabmaiti changed the base branch from develop to develop-v2 July 19, 2022 04:59
@pallabmaiti pallabmaiti mentioned this pull request Jul 19, 2022
Closed
13 tasks
@pallabmaiti pallabmaiti linked an issue Jul 19, 2022 that may be closed by this pull request
BUG: SDK crashing on bad exception #191
Closed
@pallabmaiti pallabmaiti marked this pull request as ready for review July 19, 2022 06:01
@pallabmaiti pallabmaiti changed the title [Bugfix] Race condition in iOS SDK-v2 fix: race conditions Nov 10, 2022
@1abhishekpandey 1abhishekpandey mentioned this pull request Jan 20, 2023
Merged
@pallabmaiti pallabmaiti changed the base branch from develop-v2 to master-v2 January 25, 2023 13:42
@pallabmaiti pallabmaiti changed the base branch from master-v2 to develop-v2 March 29, 2023 03:03
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 23 Code Smells

0.0% 0.0% Coverage
6.5% 6.5% Duplication

@pallabmaiti pallabmaiti marked this pull request as draft November 27, 2023 10:15
@codecov Codecov
Copy link

codecov bot commented Nov 28, 2023

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

@pallabmaiti pallabmaiti linked an issue Dec 6, 2023 that may be closed by this pull request
BUG : ControlPlaneURL is missing the path #343
Closed
@sonarqubecloud SonarQubeCloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 38 Code Smells

45.1% 45.1% Coverage
2.7% 2.7% Duplication

@pallabmaiti pallabmaiti deleted the fix/raceCondition branch December 14, 2023 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pallabmaiti pallabmaiti pallabmaiti left review comments

@itsdebs itsdebs Awaiting requested review from itsdebs

Assignees

@pallabmaiti pallabmaiti

@1abhishekpandey 1abhishekpandey

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

BUG : ControlPlaneURL is missing the path BUG: SDK crashing on bad exception
2 participants
@1abhishekpandey @pallabmaiti