fermyon / homebrew-tap

Fermyon's Homebrew Tap
Apache License 2.0
2 stars 9 forks source link

edit spin Formula test procedure to verify installed version #18

Closed dimitrilw closed 9 months ago

dimitrilw commented 10 months ago

Performs version validation after installation & closes #17 .

dimitrilw commented 10 months ago

ok... I swear I looked in my repo to confirm I'd signed the commit before making the PR...... :|

image

...not sure why DCO check failed, but following the bot's instructions & will resubmit PR. :|

dimitrilw commented 9 months ago

@kate-goldenring , I apologize for being slow. I realize I will be docked 10 free-labor dollars on my monthly salary. 😂

File steps_output.txt removed and .gitignore added so future contributors will not make the same error.

dimitrilw commented 9 months ago

I also just added a note to the Readme re running test-bot locally. I hope that out-of-scope addition is ok.

kate-goldenring commented 9 months ago

@dimitrilw thank you for updating the readme too. I found running that command locally not very helpful (despite recommending it). Did you find that you were able to specify it to test a specific, yet to be installed formula?

We also need to rebase this off latest main again. Sorry! I realized that adding testing for version is another place someone could mismatch version so i made an update in this PR to use the version variable throughout the formula which this could leverage: https://github.com/fermyon/homebrew-tap/pull/21

dimitrilw commented 9 months ago

I'll rebase pronto.

Re local run: really no idea. I ran it. It didn't complain. Neither did the Cloud-CI run. Next time cloud run fails but local passes, then I can respond with "nope; no use." 😄 ...but until then, I will go with a healthy dose of faith that it does something, even if that "something" is just a placebo effect for me.

dimitrilw commented 9 months ago

Merge was not happy. Cleaner to push as new branch; please look to PR #22 . Apologies for the mess, @kate-goldenring . ....git. 🤷🏻