Skip to content

Conversation

@dopry
Copy link
Member

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.

@doprydopry changed the title fix: npm install waa failingfix: test/app/rp npm install failingJun 12, 2024
@doprydopryforce-pushed the fix/broken-test-app-rp branch 4 times, most recently from 328c2de to 810625bCompareJune 12, 2024 14:37
@dopry
Copy link
MemberAuthor

@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

@doprydopry requested a review from n2ygkJune 12, 2024 14:41
@doprydopryforce-pushed the fix/broken-test-app-rp branch 2 times, most recently from 400bba4 to 0fe1b7fCompareJune 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.
@doprydopryforce-pushed the fix/broken-test-app-rp branch from 0fe1b7f to b6d770fCompareJune 12, 2024 15:52
@n2ygk
Copy link
Contributor

@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
Copy link
MemberAuthor

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

@n2ygkn2ygk merged commit 12236cd into django-oauth:masterJun 12, 2024
@doprydopry deleted the fix/broken-test-app-rp branch June 12, 2024 21:15
@n2ygkn2ygk added this to the 3.0.0 milestone Aug 13, 2024
Sign up for freeto 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

@dopry@n2ygk