Skip to content

Commit

Permalink
config: convert average_run_time to an integer (#291)
Browse files Browse the repository at this point in the history
There are two reasons for this change:

1. Having the average run time as a float gave an impression of
   exactness, but the actual run time varies significantly for a variety
   of reasons (e.g. how busy it is on the server). The fractional
   component would almost never actually conform to the real situation.

2. jq is often used to work with track config.json files (e.g. to
   add elements to it), and it will remove any trailing .0 fractional
   part from a number, which caused `configlet lint` to fail. Those jq
   scripts would have to work around this by manually adding .0 to it.
  • Loading branch information
ErikSchierboom authored Jul 18, 2023
1 parent 541c4c1 commit eec393e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
"highlightjs_language": "zig"
},
"test_runner": {
"average_run_time": 5.0
"average_run_time": 5
},
"files": {
"solution": [
Expand Down

0 comments on commit eec393e

Please sign in to comment.