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: test/app/rp npm install failing #1430

Merged
merged 2 commits into from
Jun 12, 2024

Conversation

dopry
Copy link
Contributor

@dopry dopry commented Jun 12, 2024

it seems we merged a major update from dependabot to some of the RP dependencies that were incompatible with some of the other package versions. This resolves that and updates the RP to Svelte4/SvelteKit 2.

Test by following the instructions in /test/app/README.md to launch the local environment.

@dopry dopry changed the title fix: npm install waa failing fix: test/app/rp npm install failing Jun 12, 2024
@dopry dopry force-pushed the fix/broken-test-app-rp branch 4 times, most recently from 328c2de to 810625b Compare June 12, 2024 14:37
@dopry
Copy link
Contributor Author

dopry commented Jun 12, 2024

@n2ygk This should be ready to merge. If you can take a quick look. I updated the Deps for the RP and added some build jobs so we'll catch if dependabot breaks builds in the RP via CI. I'd like to get this in first then I can rebase #1366

@dopry dopry requested a review from n2ygk June 12, 2024 14:41
@dopry dopry force-pushed the fix/broken-test-app-rp branch 2 times, most recently from 400bba4 to 0fe1b7f Compare June 12, 2024 15:52
it seems we merged a major update from dependabot to some of the RP
dependencies that were incompatible with some of the other package
versions. This resolves that and updates the RP to Svelte4/SvelteKit 2.

Test by following the instructions in /test/app/README.md to launch
the local environment.
@dopry dopry force-pushed the fix/broken-test-app-rp branch from 0fe1b7f to b6d770f Compare June 12, 2024 15:52
@n2ygk
Copy link
Member

n2ygk commented Jun 12, 2024

@dopry looks like this workflow failed:

Invalid workflow file: .github/workflows/test.yml#L120
The workflow is not valid. .github/workflows/test.yml (Line: 120, Col: 9): Job 'success' depends on unknown job 'test-rp'.

.github/workflows/test.yml Outdated Show resolved Hide resolved
@dopry
Copy link
Contributor Author

dopry commented Jun 12, 2024

That's what I get for doing last minute rename changes

@n2ygk n2ygk merged commit 12236cd into jazzband:master Jun 12, 2024
27 checks passed
@dopry dopry deleted the fix/broken-test-app-rp branch June 12, 2024 21:15
@n2ygk n2ygk added this to the 3.0.0 milestone Aug 13, 2024
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.

2 participants