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

Remove vendored CUE dependencies #13

Merged
merged 6 commits into from
Nov 21, 2024
Merged

Remove vendored CUE dependencies #13

merged 6 commits into from
Nov 21, 2024

Conversation

AntoineThebaud
Copy link
Contributor

@AntoineThebaud AntoineThebaud commented Nov 19, 2024

Replace the vendored CUE dependencies by a call to percli plugin update

  • for local dev env: add doc to explain how to setup things
  • for CI: new job to get the dependencies, that gets triggered before running the validation & building the plugin archives (⚠️ cue files not embedded in the plugin archives yet)

@AntoineThebaud AntoineThebaud changed the title Antoinethebaud/use percli CI: add step to get dependencies for plugin schemas Nov 19, 2024
@AntoineThebaud AntoineThebaud force-pushed the antoinethebaud/use-percli branch 3 times, most recently from 8727c5f to 8d16a0a Compare November 19, 2024 19:47
@AntoineThebaud AntoineThebaud changed the title CI: add step to get dependencies for plugin schemas Remove vendored CUE dependencies Nov 19, 2024
@AntoineThebaud AntoineThebaud marked this pull request as ready for review November 19, 2024 20:20
@AntoineThebaud AntoineThebaud marked this pull request as draft November 19, 2024 20:20
Antoine THEBAUD added 6 commits November 20, 2024 15:21
@AntoineThebaud AntoineThebaud marked this pull request as ready for review November 20, 2024 14:28
Copy link
Member

@Nexucis Nexucis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice !

@Nexucis Nexucis merged commit f1c1243 into main Nov 21, 2024
6 checks passed
@Nexucis Nexucis deleted the antoinethebaud/use-percli branch November 21, 2024 09:26
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