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 JavaLoggingClient.java #387

Closed
wants to merge 1 commit into from
Closed

Update JavaLoggingClient.java #387

wants to merge 1 commit into from

Conversation

alexeagle
Copy link
Member


Changes are visible to end-users: yes/no

  • Searched for relevant documentation and updated as needed: yes/no
  • Breaking change (forces users to change their own code or config): yes/no
  • Suggested release notes appear below: yes/no

Test plan

  • Covered by existing test cases
  • New test cases added
  • Manual testing; please provide instructions so we can reproduce:

Copy link

aspect-workflows bot commented Jan 8, 2025

Test

⚠️ Buildkite build #352 failed.

//logger/client/src/build/aspect:JavaLoggingClient failed to build

worker spawn failed for Javac
 
warning: [options] source value 8 is obsolete and will be removed in a future release
warning: [options] target value 8 is obsolete and will be removed in a future release
warning: [options] To suppress warnings about obsolete options, use -Xlint:-options.
logger/client/src/build/aspect/JavaLoggingClient.java:12: error: ';' expected
    Scanner scanner = ne Scanner(System.in);
                        ^

💡 To reproduce the build failures, run

bazel build //logger/client/src/build/aspect:JavaLoggingClient

Format

Formatting check has failed

💡 Some formatting failures can be fixed automatically by running the command below, while others may require manual fixes

bazel run //:format

Lint

@alexeagle alexeagle closed this Jan 8, 2025
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.

1 participant