fix: test/app/rp npm install failing

jazzband / django-oauth-toolkit

OAuth2 goodies for the Djangonauts!

https://django-oauth-toolkit.readthedocs.io

Other

3.13k stars 792 forks source link

fix: test/app/rp npm install failing #1430

Closed dopry closed 3 months ago

dopry commented 3 months ago

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 commented 3 months ago

@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 https://github.com/jazzband/django-oauth-toolkit/pull/1366

n2ygk commented 3 months ago

@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'.

dopry commented 3 months ago

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