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

Mok liee patch 2 #757

Closed
wants to merge 9 commits into from
Closed

Mok liee patch 2 #757

wants to merge 9 commits into from

Conversation

wing328
Copy link
Member

@wing328 wing328 commented Mar 3, 2024

No description provided.

@mok-liee
Copy link
Contributor

mok-liee commented Mar 3, 2024

great, i was trying to do this in the same build workflow with conditions. i created a new branch on my fork to keep my tryings away from the PR.
https://github.com/mok-liee/openapi-generator-cli/tree/feature/add-windows-to-build

which way is preferable? split workflow or add conditions?

@wing328 wing328 closed this Mar 3, 2024
@wing328
Copy link
Member Author

wing328 commented Mar 3, 2024

let me think about it first. please put the work on hold for the time being

@wing328 wing328 deleted the mok-liee-patch-2 branch March 3, 2024 10:34
@wing328
Copy link
Member Author

wing328 commented Mar 3, 2024

the enhancement to support JAVA_HOME variable has been included in the latest release.

please give it a try.

thanks again for the PR.

@mok-liee
Copy link
Contributor

mok-liee commented Mar 3, 2024

okay, let me know when i can help. i think i'll try a bit in my fork to learn more about github actions.

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.

None yet

3 participants