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

Logging #4

Open
BraeWebb opened this issue Jun 15, 2019 · 1 comment
Open

Logging #4

BraeWebb opened this issue Jun 15, 2019 · 1 comment
Assignees
Milestone

Comments

@BraeWebb
Copy link
Owner

Chalkbox tells you nothing about what it's doing and this is going to need to change ASAP.

We'll use a logging library with varying levels and add command line flags to specify the level.

  • SEVERE: level is on by default, occurs if there is an error that might prevent a process from running correctly, for example sample solution not compiling.
  • WARNING: logs errors that occur for individual students (the edge cases) for example an IOException when trying to run their assignment or something else weird that would need further investigation.
  • INFO: Describes the broad strokes of what chalkbox is doing. For example, printing out that each student is collected and that a process is running.
  • FINE: Describes when the individual steps of a chalkbox process occur. For example, may inform users that a submission is being compiled, then JUnit tests running, etc.
  • FINER: Describes the output of a submission being run. For example, the tests that pass or fail.
  • FINEST: A dump of almost all information that chalkbox has access to that might be useful for debugging. Intended to only be run on some problem submissions.
@wisebaldone
Copy link
Collaborator

Dependency Injection should be used to present the logger to each class that needs it.

@BraeWebb BraeWebb added this to the Release v2.0 milestone Dec 17, 2019
@BraeWebb BraeWebb mentioned this issue Dec 17, 2019
4 tasks
wisebaldone pushed a commit that referenced this issue Apr 11, 2020
Added Loggable interface with default method for #4.
Added picocli to give ability for flags and fleshed out a bunch of skeletons for the sub commands.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants