Skip to content

Revert "specific usePropsFor to treeshake out unncessary calling/chat code"#5728

Merged
vhuseinova-msft merged 1 commit intomainfrom
revert-5684-alkwa/create-modality-specific-selectors
Mar 20, 2025
Merged

Revert "specific usePropsFor to treeshake out unncessary calling/chat code"#5728
vhuseinova-msft merged 1 commit intomainfrom
revert-5684-alkwa/create-modality-specific-selectors

Conversation

@vhuseinova-msft
Copy link
Copy Markdown
Member

Reverts #5684

@github-actions
Copy link
Copy Markdown
Contributor

CallWithChat bundle size is not changed.

  • Current size: 12401048
  • Base size: 12401048
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

Calling bundle size is not changed.

  • Current size: 12401036
  • Base size: 12401036
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

Chat bundle size is not changed.

  • Current size: 1775228
  • Base size: 1775228
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@azure/communication-react jest test coverage for stable.

Lines Statements Functions Branches
Base 29294 / 45995
63.68%
29294 / 45995
63.68%
808 / 1476
54.74%
2378 / 3808
62.44%
Current 29299 / 45995
63.7%
29299 / 45995
63.7%
808 / 1476
54.74%
2404 / 3820
62.93%
Diff 5 / 0
0.02%
5 / 0
0.02%
0 / 0
0%
26 / 12
0.49%

@github-actions
Copy link
Copy Markdown
Contributor

@azure/communication-react jest test coverage for beta.

Lines Statements Functions Branches
Base 58428 / 94462
61.85%
58428 / 94462
61.85%
1180 / 2698
43.73%
3552 / 5839
60.83%
Current 58470 / 94462
61.89%
58470 / 94462
61.89%
1180 / 2698
43.73%
3530 / 5827
60.58%
Diff 42 / 0
0.04%
42 / 0
0.04%
0 / 0
0%
-22 / -12
-0.25%

@vhuseinova-msft vhuseinova-msft merged commit db756c3 into main Mar 20, 2025
54 of 78 checks passed
@vhuseinova-msft vhuseinova-msft deleted the revert-5684-alkwa/create-modality-specific-selectors branch March 20, 2025 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change does not need changelog Changes that does not affect the published package in any way do not need changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants