flagon-io/hello

public

Greets people from the command line. The sample repository for g1t.

Add a Makefile with build, test and run targets #54

Mergedg1t merged into maing1t-agenton g1t

Approved by g1t-agent

ResolvedAdd a Makefile with build, test and run targets#52

Merged into main by g1t automatically, once it was ready, · f7e69cd

syntaqxopened this pull request with g1t-agent

I added a Makefile at the repo root with build, test and run targets. build is the default goal, all three are declared .PHONY, and the recipes are tab-indented. run runs cargo run -- $(ARGS), so you can pass arguments with make run ARGS="--shout Ada".

I ran make build, make test and make run ARGS=Ada. All three passed, and the run printed Hi, Ada!. I didn't try --shout or any other flags through ARGS.

No other files changed, so this shouldn't conflict with the open pull requests. I didn't open any issues or comment on other pull requests.

syntaqxmarked this ready for review·
g1trequested a review from g1t-agent·
g1t-agentapproved these changes

Makefile matches issue #52: build is the default goal, all three targets are .PHONY, recipes are tab-indented, and only the Makefile is added. I ran make build (ok) and make run ARGS="--shout Ada", which printed HI, ADA!, so the untested --shout path works. No problems found.

Reviewed by a g1t agent on Claude Sonnet 5.5.

g1tadded this to the merge queue·
g1tmerged this·

Sign in to comment.