Skip to content

feat: convenience script to run any class's main #808

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.

Sign up for GitHub

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

Jump to bottom
Merged
baltzell merged 2 commits into from
Sep 9, 2025
Merged

feat: convenience script to run any class's main #808

baltzell merged 2 commits into from
Sep 9, 2025

Conversation

@c-dilks
Copy link
Member

@c-dilks c-dilks commented Sep 2, 2025

No description provided.

@c-dilks
Copy link
Member Author

c-dilks commented Sep 2, 2025

For now I just named it coatjava, but better name suggestions are certainly welcome.

@c-dilks c-dilks mentioned this pull request Sep 2, 2025
Merged
baltzell
baltzell previously approved these changes Sep 3, 2025
View reviewed changes
@baltzell
Copy link
Collaborator

baltzell commented Sep 4, 2025

  • coatjava
  • coatjavajava
  • run-java (don't we have other "run-standard-thing" wrappers?)

@c-dilks
Copy link
Member Author

c-dilks commented Sep 5, 2025

  • run-java (don't we have other "run-standard-thing" wrappers?)

run-java would follow our convention (run-groovy, run-jshell, etc.) and I think it wouldn't conflict with anything else in the typical $PATH. I decided on run-coatjava though, since run-java sounds a bit too generic...

Unfortunately we're stuck with other generic names, like run-groovy; I'd rather them be run-coatjava-groovy or something, but downstream Groovy users may depend on run-groovy. We could phase out the generic names with name-change warnings.

@c-dilks c-dilks requested a review from baltzell September 5, 2025 14:38
@baltzell baltzell enabled auto-merge (squash) September 9, 2025 21:53
@baltzell baltzell merged commit 344cd31 into development Sep 9, 2025
33 checks passed
@baltzell baltzell deleted the main-wrapper branch September 9, 2025 21:55
c-dilks added a commit that referenced this pull request Sep 15, 2025
@c-dilks
feat: convenience script to run any class's main (#808)
572db3b 
* feat: convenience script to run any class's `main`

* refactor!: rename and generalize usage guidance
tongtongcao pushed a commit that referenced this pull request Sep 22, 2025
@c-dilks @tongtongcao
feat: convenience script to run any class's main (#808)
e1caccd 
* feat: convenience script to run any class's `main`

* refactor!: rename and generalize usage guidance
tongtongcao pushed a commit that referenced this pull request Nov 18, 2025
@c-dilks @tongtongcao
feat: convenience script to run any class's main (#808)
6d756b2 
* feat: convenience script to run any class's `main`

* refactor!: rename and generalize usage guidance
tongtongcao pushed a commit that referenced this pull request Nov 21, 2025
@c-dilks @tongtongcao
feat: convenience script to run any class's main (#808)
c92b805 
* feat: convenience script to run any class's `main`

* refactor!: rename and generalize usage guidance
tongtongcao pushed a commit that referenced this pull request Dec 5, 2025
@c-dilks @tongtongcao
feat: convenience script to run any class's main (#808)
74e9f57 
* feat: convenience script to run any class's `main`

* refactor!: rename and generalize usage guidance
tongtongcao pushed a commit that referenced this pull request Dec 5, 2025
@c-dilks @tongtongcao
feat: convenience script to run any class's main (#808)
01a6c8f 
* feat: convenience script to run any class's `main`

* refactor!: rename and generalize usage guidance
tongtongcao pushed a commit that referenced this pull request Dec 31, 2025
@c-dilks @tongtongcao
feat: convenience script to run any class's main (#808)
3373c01 
* feat: convenience script to run any class's `main`

* refactor!: rename and generalize usage guidance
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@baltzell baltzell baltzell approved these changes

@raffaelladevita raffaelladevita Awaiting requested review from raffaelladevita raffaelladevita is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@c-dilks @baltzell