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

fix: package manager configs #1415

Merged
merged 7 commits into from
Nov 8, 2023
Merged

Conversation

arboleya
Copy link
Member

@arboleya arboleya commented Nov 8, 2023

Copy link
Contributor

github-actions bot commented Nov 8, 2023

This PR is published in NPM with version 0.0.0-pr-1415-20231108102922

Copy link
Contributor

github-actions bot commented Nov 8, 2023

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 85.96% 5731/6667
🟡 Branches 69.17% 846/1223
🟡 Functions 75.71% 932/1231
🟢 Lines 85.98% 5475/6368

Test suite run success

1368 tests passing in 248 suites.

Report generated by 🧪jest coverage report action from 7d54c56

nedsalk
nedsalk previously approved these changes Nov 8, 2023
@arboleya arboleya marked this pull request as draft November 8, 2023 10:46
@nedsalk nedsalk self-requested a review November 8, 2023 10:47
@nedsalk nedsalk dismissed their stale review November 8, 2023 10:48

'twas a hasty approval

@arboleya arboleya marked this pull request as ready for review November 8, 2023 11:59
@arboleya arboleya enabled auto-merge (squash) November 8, 2023 12:00
nedsalk
nedsalk previously approved these changes Nov 8, 2023
@arboleya arboleya disabled auto-merge November 8, 2023 12:12
@arboleya arboleya merged commit 05c0492 into master Nov 8, 2023
8 checks passed
@arboleya arboleya deleted the aa/fix/package-manager-configs branch November 8, 2023 12:41
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.

Remove pnpm requirement for SDK consumers
4 participants