feat: add noddy formula#13
Merged
Merged
Conversation
noddy is a macOS command line tool collection for DevOps, published from clouddrove/noddy. The formula tracks the v1.0.0 tag. It is a shell tool rather than a compiled binary, so there is a single source tarball rather than per-architecture archives. The script and its toyland/ payload install together into libexec, with only the entry point linked onto PATH, because noddy sources its plugins from a path relative to itself. Completions for zsh, bash and fish are installed from the tarball.
The tap had no CI, so a formula could be merged without anyone confirming it installs. Add a workflow that audits, installs from source and runs the test block for every formula a change touches. Scoped to changed files deliberately: installing every formula on every push would fail on unrelated pre-existing issues and would say nothing about the change under review. brew audit, install and test take a formula name rather than a path, and passing a path to audit is disabled outright, so the checkout is registered as the tap first and the formulae referred to by name. Also fixes the noddy formula, which this workflow would have caught: libexec.install moves toyland/ rather than copying it, so the completion files no longer existed by the time they were installed and the build failed. Completions are now installed first. Four brew audit --strict findings are fixed alongside: a redundant version, 'command line' in the description, and a string interpolation in the test block.
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.
Adds a formula for clouddrove/noddy, tracking the new v1.0.0 release.
Why
noddy was previously installable only via
curl | shfrom the default branch, which fetches unversioned, unchecksummed code. This gives it a tagged, SHA-256 verified install path.Formula notes
Unlike
smurfandkuconf, noddy is a shell tool, so there is a single source tarball rather than per-architecture archives.The script and its
toyland/payload install together intolibexec, with only the entry point symlinked onto PATH, because noddy sources its plugins from a path relative to itself. noddy resolves symlinks before locating that payload, so the symlink works.Completions for zsh, bash and fish ship from the tarball.
Verified
Install steps and the test block were simulated locally against the released tarball:
noddy helpandnoddy categoriesproduce expected output through thebinsymlinknoddy infoexits 07824218f...4463matches the release tarballbrew install --build-from-sourcehas not been run against this branch; worth a check before merge.