Skip to content

Workflow generation extension and CLI#381

Open
iocanel wants to merge 1 commit into
quarkiverse:mainfrom
iocanel:workflow-generator
Open

Workflow generation extension and CLI#381
iocanel wants to merge 1 commit into
quarkiverse:mainfrom
iocanel:workflow-generator

Conversation

@iocanel

@iocanel iocanel commented May 13, 2025

Copy link
Copy Markdown

The pull request introduces a mini extension that can be used to automatically generate a github action.
Additionally, it introduces a CLI module that can now be used generate the workflow without needing to add any extension to the mix.

@iocanel iocanel requested a review from a team as a code owner May 13, 2025 13:35
@iocanel iocanel force-pushed the workflow-generator branch from 3042a65 to a852a2c Compare May 14, 2025 09:03

@gsmet gsmet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Am I right that it's not related at all to what this extension is currently doing (i.e. allow developing GitHub Actions with Quarkus)?

Asking because I'm not completely sure we should tie the two together and maybe this should be hosted in another repository.

I also spotted a few issues that you will want to fix, whether we end up including this code here or elsewhere.

"quarkus-github-action-workflow-generator", null,
"jar", GenerateCommand.getVersion());

@Parameters(arity = "0..1", paramLabel = "GENERATION_PATH", description = " The path to generate the Helm Charts")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The description looks off.

@@ -0,0 +1,3 @@
quarkus.log.level=ERROR
quarkus.log.category."com.fasterxml.jackson".level=DEBUG

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks suspicious.

Comment thread deployment/pom.xml Outdated
Comment on lines +36 to +39
<dependency>
<groupId>io.quarkus</groupId>
<artifactId>quarkus-devtools-common</artifactId>
</dependency>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm puzzled you have to adjust the original extension?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I don't. Left overs after moving things around. Removing it.

@iocanel iocanel force-pushed the workflow-generator branch from a852a2c to a23722d Compare May 14, 2025 13:05
@iocanel

iocanel commented May 14, 2025

Copy link
Copy Markdown
Author

@gsmet: Applied your feedback. Also, you are right. This feature has nothing to do with the existing functionality it's just that both are related to github actions. I want to avoid having multiple different extension related to github actions, so I added the extension here. I am open to move the code elsewhere if it makes more sense.

We briefly discussed that with @maxandersen, too. wdyt ?

@iocanel iocanel requested a review from gsmet May 22, 2025 12:03
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