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

Update README.md: no java HelloWorldApp.java #611

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

davidkhala
Copy link

@davidkhala davidkhala commented Mar 22, 2024

A confusion introduced by running java HelloWorldApp.java

I believe the file is not an artifact of setup-java, thus user should provide it. But there is no tips in the guide that user should prepare it in advance.

Directly copy the code snippets will introduce a straightforward error, which should not happen in a guide.

A confusion introduced by running `java HelloWorldApp.java`
I believe the file is not an artifact of `setup-java`, thus user should provide it. But there is no tips in the guide that user should prepare it in advance. Directly copy the code snippets will introduce a straightforward error, which should not happen in a guide.
@davidkhala davidkhala requested a review from a team as a code owner March 22, 2024 03:47
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

1 participant