Skip to content

feat: static generated library #2

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

Merged
merged 7 commits into from
Jul 8, 2024
Merged

feat: static generated library #2

merged 7 commits into from
Jul 8, 2024

Conversation

Duologic
Copy link
Contributor

@Duologic Duologic commented Jun 30, 2024

This generates another static version of this library following the pattern I've used successfully in other libraries.

It doesn't include the reusableWorkflowCallJob part, not sure yet what that is used for. I'm new to GitHub Actions.

The generator depends on crdsonnet/crdsonnet#24

@Duologic Duologic requested a review from julienduchesne June 30, 2024 19:40
+ {
job+: {
steps:: {},
step: (super.steps + { '#'+: { name: 'step' } }),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just noticed that this doesn't work with the language server. 🤷

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Started a PR with testcases for this here: grafana/jsonnet-language-server#144

@julienduchesne
Copy link
Contributor

Have you found a use for jsonnet -> github actions workflows? I have made some fairly complex workflows in GHA and haven't found the need yet. The matrix feature is pretty powerful. Drone needed it because the matrix stuff was terrible

Copy link
Contributor

@julienduchesne julienduchesne left a comment

Choose a reason for hiding this comment

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

:shipit:

@Duologic Duologic merged commit 15dc64c into main Jul 8, 2024
@Duologic Duologic deleted the duologic/static branch July 8, 2024 09:46
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