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

CI .npmrc temporary CI setting #25

Merged
merged 10 commits into from
Oct 27, 2022
Merged

CI .npmrc temporary CI setting #25

merged 10 commits into from
Oct 27, 2022

Conversation

Omri-Levy
Copy link
Contributor

added strict-peer-dependencies=false temporarily to avoid breaking vite and vite-plugin-svelte

@changeset-bot
Copy link

changeset-bot bot commented Oct 27, 2022

🦋 Changeset detected

Latest commit: 28937e8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
web-sdk Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

…is setup for ci

currently breaks due to origin and head, requires special config.
@Omri-Levy Omri-Levy merged commit 7506870 into main Oct 27, 2022
MihirMandviwala pushed a commit that referenced this pull request Feb 12, 2024
* chore(version control): init commit for opening a wip pr

* refactor(.github): moved web-sdk's github actions to root level

all actions now use pnpm instead of yarn, ci now includes a build step.

* ci(.npmrc): added auto-install-peers = true

prevents pnpm install erroring in CI, could potentially be used inline in CI only

* fix(.npmrc): making sure .npmrc is versioned and its rules apply

* refactor(changesets): moved the dependencies and the required scripts to root for changesets

* ci(wip peer deps fix): trying to fix CI only issue of peer deps

* fix(peer deps for ci): temporarly added strict-peer-dependencies=false to avoid breaking changes

fails CI without the .npmrc flag

* revert(nx affected): temporarily reverting to run-all until affected is setup for ci

currently breaks due to origin and head, requires special config.
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.

None yet

1 participant