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): pull release/v1.87.0 into main #3937

Merged
merged 7 commits into from
Dec 16, 2024
Merged

Conversation

devops-github-rudderstack
Copy link
Contributor

👑 An automated PR

koladilip and others added 7 commits December 10, 2024 11:16
* refactor: code to address sonar issues

* refactor: garl record transform
* fix: handling partial error in a batch for reddit destination

* fix: minor change

* fix: moving networkHandler to v1 from v0

* fix: sonarcloud code analysis
* chore: topsort changes

* chore: boilerplates

* chore: topsort changes

* chore: function updates

* chore: product array changes

* chore: purchase event implementation

* chore: purchase function updated

* chore: build finalPayload

* chore: few updations

* chore: mockfn changes

* chore: restructured files
- moved impressions and clicks logic into its own file
- moved purchase related logic into its own file

* chore: update formatting

* chore: test cases added

* chore: destType updated

---------

Co-authored-by: Utsab Chowdhury <[email protected]>
Co-authored-by: Sai Sankeerth <[email protected]>
@devops-github-rudderstack
Copy link
Contributor Author

Copy link

codecov bot commented Dec 13, 2024

Codecov Report

Attention: Patch coverage is 94.85981% with 11 lines in your changes missing coverage. Please review.

Project coverage is 90.50%. Comparing base (0965f30) to head (7d4fbaa).

Files with missing lines Patch % Lines
src/v0/destinations/topsort/transform.js 89.23% 7 Missing ⚠️
src/v0/destinations/topsort/utils.js 89.28% 3 Missing ⚠️
src/v0/util/index.js 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3937      +/-   ##
==========================================
+ Coverage   90.48%   90.50%   +0.01%     
==========================================
  Files         615      620       +5     
  Lines       32359    32518     +159     
  Branches     7687     7710      +23     
==========================================
+ Hits        29281    29429     +148     
- Misses       2822     2865      +43     
+ Partials      256      224      -32     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ItsSudip ItsSudip merged commit 9d6a002 into main Dec 16, 2024
30 checks passed
@ItsSudip ItsSudip deleted the release/v1.87.0 branch December 16, 2024 04:41
@@ -84,14 +84,13 @@ export const v0oauthScenarios = [
status: 401,
body: {
output: {
authErrorCategory: 'REFRESH_TOKEN',
Copy link
Collaborator

Choose a reason for hiding this comment

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

test case name should be updated

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.

9 participants