Skip to content

Conversation

@DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented Nov 30, 2025

Currently, we test the following downstream packages:

And we can easily add more.

The motivation here is to catch breakages (example GunnarFarneback/LocalRegistry.jl#99) sooner.

@DilumAluthge DilumAluthge marked this pull request as draft November 30, 2025 18:41
@DilumAluthge DilumAluthge marked this pull request as ready for review November 30, 2025 18:52
@GunnarFarneback
Copy link
Collaborator

Nice. We want RegistryCI too here, right?

@DilumAluthge
Copy link
Member Author

Nice. We want RegistryCI too here, right?

Good call. I've added it.

@DilumAluthge DilumAluthge force-pushed the dpa/downstream-ci branch 2 times, most recently from 9380a26 to a3cdd94 Compare November 30, 2025 19:35
@DilumAluthge
Copy link
Member Author

Alright, I think this is all set now.

We currently test against Registrator.jl, RegistryCI.jl, and LocalRegistry.jl. Any other downstream packages we want to add?

@GunnarFarneback
Copy link
Collaborator

It looks like it does what it should but I have no competence to review GitHub actions.

@DilumAluthge
Copy link
Member Author

Maybe I can recruit someone to review the GitHub Actions details. @giordano maybe?

DilumAluthge and others added 3 commits November 30, 2025 18:00
It should be fine to `Pkg.activate()` with a trailing slash.

Co-authored-by: Mosè Giordano <[email protected]>
Co-authored-by: Mosè Giordano <[email protected]>
@DilumAluthge
Copy link
Member Author

Thank you @giordano!!

@DilumAluthge
Copy link
Member Author

@GunnarFarneback Shall we give this a go?

@giordano
Copy link
Member

giordano commented Dec 5, 2025

Bump?

@DilumAluthge DilumAluthge merged commit a4f785e into master Dec 5, 2025
31 of 32 checks passed
@DilumAluthge DilumAluthge deleted the dpa/downstream-ci branch December 5, 2025 14:49
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.

4 participants