-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat: Add PHP template #63
Conversation
4b8a094
to
841d270
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice PR! Thanks for the contrib. I'll approve the CI job to check this builds, just one nitpicky request for changes :)
841d270
to
fb148da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Will get this merged once I verify that it actually builds, CI is having... issues currently lol
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, so. When trying to build this, it seems this doesn't output anything resembling a binary or script in the /bin
path in the derivation output.
/tmp/tmp.mUjAv72oFJ via 🐘
❯ nix run .
error: unable to execute '/nix/store/pn5ag15jq0kkrw6iwd36hh74wwak9q5j-example-0.1.0/bin/example': No such file or directory
If this won't create any form of entrypoint executable, that's fine, but we'll drop the package
entries for it and make it a dev shell-only one. Unless you can think of a better implementation here?
Happy for you to implement this as a binary, but I'll not merge this in its current state as it 'builds' but has nothing to run.
We can chat more about it in Discord if I've been unclear at all here.
1709db8
to
8f0af67
Compare
8f0af67
to
6ee1130
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I'll re-review later on :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All works locally! Sorry for the delay @Sironheart :)
No description provided.