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

run output improvements #32

Merged
merged 8 commits into from
Jul 5, 2023
Merged

Conversation

MasloMaslane
Copy link
Member

@MasloMaslane MasloMaslane commented Jun 21, 2023

  • Alternative time/memory units in config #26
  • replace Time limit (in ms): 1000 with Time limit: 1000 ms, Memory limit (in kb): 16000 with Memory limit: 16000 kB (note also that probably it should be kB instead of kb,
  • replace Task Simple test package (abc) with Task: Simple test package (tag: abc),
  • list (e.g. in a single line) the tests for which the solutions will be run,
  • after printing the list, remind the user about how to regenerate tests and that run runs on the already generated tests,
  • give a warning when run runs only on sample test,
  • give an error when run runs on 0 tests,

@MasloMaslane MasloMaslane self-assigned this Jun 21, 2023
@MasloMaslane MasloMaslane linked an issue Jun 21, 2023 that may be closed by this pull request
7 tasks
@MasloMaslane MasloMaslane changed the title 27 wip cli run output improvements run output improvements Jun 22, 2023
@tonowak
Copy link
Collaborator

tonowak commented Jul 1, 2023

It's not yet ready for review, correct?

@tonowak
Copy link
Collaborator

tonowak commented Jul 1, 2023

I've reviewed the current code. We can merge it and implement the last feature in another PR.

@MasloMaslane MasloMaslane marked this pull request as ready for review July 5, 2023 08:01
@MasloMaslane MasloMaslane merged commit 074fab6 into main Jul 5, 2023
@MasloMaslane MasloMaslane deleted the 27-wip-cli-run-output-improvements branch July 5, 2023 08:03
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.

CLI run output improvements
2 participants