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

chore(release): pulling release/1.21.3 into master #391

Merged
merged 5 commits into from
Feb 8, 2024

Conversation

devops-github-rudderstack
Copy link
Contributor

👑 An automated PR

itsdebs and others added 5 commits January 31, 2024 14:53
chore(release): pulling main into develop post release v1.21.2
…e serializing any object (#382)

* fix: removed invalid numbers if found while serializing RudderMessage object

* chore: minor improvements while removing invalid numbers from message

* fix: stringifying special floating numbers instead of filtering them out

* chore: fixed sonarcloud issues

* chore: renamed dump to process across the codebase

* fix: fixed serializing of special floating point numbers in RudderContext and RudderTraits Adapter as well

* fix: fixed failing unit tests for invalid numbers

* chore: minor improvements

---------

Co-authored-by: Desu Sai Venkat <[email protected]>
* fix: race condition fix using semaphore

* fix: sonarlint issues

* fix: acquire and wait
* fix: fixed serialization of strings twice

* test: added unit tests for string conversion to map and list

* chore: minor improvements

* chore: added comments about the new public functions in Utils

* chore: minor improvements
Copy link

sonarqubecloud bot commented Feb 7, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@1abhishekpandey 1abhishekpandey merged commit ddac91d into master Feb 8, 2024
13 checks passed
@1abhishekpandey 1abhishekpandey deleted the release/1.21.3 branch February 8, 2024 11:15
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.

5 participants