Skip to content

fix(collab): add optional chaining for tree shaking#431

Merged
hiteshshetty-dev merged 2 commits intostage_v3from
VE-6102
Apr 23, 2025
Merged

fix(collab): add optional chaining for tree shaking#431
hiteshshetty-dev merged 2 commits intostage_v3from
VE-6102

Conversation

@ZuhairAhmed-cs
Copy link
Copy Markdown
Contributor

@ZuhairAhmed-cs ZuhairAhmed-cs commented Apr 23, 2025 •
edited by atlassian Bot
Loading

Ref :- VE-6102

@ZuhairAhmed-cs ZuhairAhmed-cs self-assigned this Apr 23, 2025
@ZuhairAhmed-cs ZuhairAhmed-cs requested a review from a team as a code owner April 23, 2025 06:52
hiteshshetty-dev
hiteshshetty-dev requested changes Apr 23, 2025
View reviewed changes
Comment thread src/index.ts Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2025 •
edited
Loading

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 70.1% 8157 / 11635
🔵 Statements 70.1% 8157 / 11635
🔵 Functions 68.03% 281 / 413
🔵 Branches 84.57% 987 / 1167
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/index.ts 38.88% 50% 10% 38.88% 17-22, 25-33, 36-37, 40-41, 44-45, 48-58, 61-62, 66, 69-70, 77
src/logger/logger.ts 100% 100% 100% 100%
src/visualBuilder/utils/collabUtils.ts 73.05% 84.21% 71.42% 73.05% 35, 85-108, 163-165, 174-201, 204-206, 294, 296-297, 300, 302-303
Generated in workflow #394 for commit 26e63f9 by the Vitest Coverage Report Action

hiteshshetty-dev
hiteshshetty-dev approved these changes Apr 23, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@hiteshshetty-dev hiteshshetty-dev left a comment

Choose a reason for hiding this comment

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

LGTM!!

@hiteshshetty-dev hiteshshetty-dev merged commit 918b374 into stage_v3 Apr 23, 2025
9 checks passed
@hiteshshetty-dev hiteshshetty-dev deleted the VE-6102 branch April 23, 2025 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hiteshshetty-dev hiteshshetty-dev hiteshshetty-dev approved these changes

@faraazb faraazb Awaiting requested review from faraazb

Assignees

@ZuhairAhmed-cs ZuhairAhmed-cs

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ZuhairAhmed-cs @hiteshshetty-dev @faraazb-contentstack