Conversation
The ordering SPA and the local development problem are no longer sent to candidates, so remove them along with the example project that only problem 3 referenced. The dependabot config existed solely to watch problem-3/client, so it goes too.
Miljar
force-pushed
the
discounts-console-command
branch
from
September 18, 2026 12:13
526be0d to
4c3d2a0
Compare
Candidates now build two ways of calling the discount service instead of one. The command prices a set of orders and reports what each one gets.
Miljar
force-pushed
the
discounts-console-command
branch
from
September 18, 2026 12:15
4c3d2a0 to
b1f67c1
Compare
BrechtBonte
approved these changes
Sep 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two changes to the candidate-facing test.
Only the discounts problem remains
The ordering SPA and the local development problem are no longer sent out, so this removes
2-ordering.md,3-local-development.md, and theproblem-3/example project that only thelatter referenced.
.github/dependabot.ymlwent with them, since it existed solely to watchproblem-3/clientfor npm updates. That also clears the 55 dependabot alerts currently openagainst master, all of which came from
problem-3/client/package-lock.json.data/andexample-orders/stay. The discounts problem uses both.The test now asks for two callers
1-discounts.mdpreviously described a single HTTP door. It now also asks for a command linecaller that prices a set of orders and reports what each one gets.
Both doors are described neutrally, with no guidance on how to build either. How a candidate
gets there, including whether they reach for a framework, is left open and asked about in the
interview instead.
Nothing else in the problem changed. Same three rules, same fixtures, same guidelines.
Reviewer note: keep this description and the commit messages free of what the added caller is
meant to reveal. The repo is public and candidates read it. That reasoning belongs in the
interviewer guide.