Skip to content
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

13 bart formula #15

Merged
merged 6 commits into from
Oct 25, 2023
Merged

13 bart formula #15

merged 6 commits into from
Oct 25, 2023

Conversation

dimitrilw
Copy link
Contributor

Should fix #13 . Also uses a test routine that validates installed version number.

@itowlson
Copy link

@dimitrilw Could you sign the commits for DCO purposes please? You'll probably also need to GPG-sign them (I'm not sure about this repo but a lot of our repos require verified GPG-signed commits). Thanks!

Copy link
Collaborator

@kate-goldenring kate-goldenring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding this in! Just a quick fix and signoff on commits and should be good

Formula/bart.rb Outdated Show resolved Hide resolved
@dimitrilw
Copy link
Contributor Author

@dimitrilw Could you sign the commits for DCO purposes please? You'll probably also need to GPG-sign them (I'm not sure about this repo but a lot of our repos require verified GPG-signed commits). Thanks!

I'll get that set-up today.

@dimitrilw
Copy link
Contributor Author

Thank you for advising me to add GPG verification. Done.

image

@dimitrilw
Copy link
Contributor Author

I see that the DCO req is for every merged commit. The "Details" advises me to rebase & resubmit. Doing that now.

image

I apologize for the Github-noob error. I have not used this feature of Github before (as you can tell) and I'm learning as I go.

@dimitrilw
Copy link
Contributor Author

dimitrilw commented Oct 20, 2023

Done.

@dimitrilw
Copy link
Contributor Author

I see the fails & I'm working to fix. Thank you for being patient with me -- first Homebrew Formula submission, so it is a learning experience.

@dimitrilw
Copy link
Contributor Author

Updated.

@dimitrilw
Copy link
Contributor Author

Also re-updated GPG signature on last commit -- I apologize. I did not realize that I needed to enable git signatures & set as default behavior in:

  • git CLI
  • VS-Code git
  • the git extension I use in VS-Code

Adding that note here in case it helps anybody else.

@dimitrilw
Copy link
Contributor Author

dimitrilw commented Oct 24, 2023

Gah! I apologize. My VS-Code settings are adding that last LF when I commit & push. Manually fixing outside VS-Code. ...I hope I have entertained you all with these errors. 😊

@dimitrilw
Copy link
Contributor Author

Of note: I do not understand the other fail on MacOS-13. It shows:
image

@kate-goldenring
Copy link
Collaborator

Thanks for all the iterations here @dimitrilw! Do you mind signing your latest commit? My assumption is that the macos workflow there was canceled because the ubuntu workflow errored but i have not dug into it

@dimitrilw
Copy link
Contributor Author

gah! I thought I had git config'd to auto-sign when I did git commit from CLI. ......so many potholes on this road. 😄

- add initial bart.rb placeholder file
- edit readme to ref bart formula

NOTE:
This Homebrew Formula is not complete.

TODO:
After binary releases are built for Mac/Linux X Intel/ARM,
then un-comment lines & update 'url' & 'sha256' values.

Signed-off-by: Dimitri <5898931+dimitrilw@users.noreply.github.com>
Signed-off-by: Dimitri <5898931+dimitrilw@users.noreply.github.com>
Signed-off-by: Dimitri <5898931+dimitrilw@users.noreply.github.com>
Signed-off-by: Dimitri <5898931+dimitrilw@users.noreply.github.com>
Signed-off-by: Dimitri <5898931+dimitrilw@users.noreply.github.com>
Signed-off-by: Dimitri <5898931+dimitrilw@users.noreply.github.com>
@dimitrilw
Copy link
Contributor Author

Done. Thank you for being patient with me.

@itowlson
Copy link

Thank you @dimitrilw for having the patience to keep at it despite all those potholes!

Copy link
Collaborator

@kate-goldenring kate-goldenring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @dimitrilw for working through all the bot suggestions! I created an issue to figure out how to add windows in the future: #16

@kate-goldenring kate-goldenring merged commit 3deb797 into fermyon:main Oct 25, 2023
3 checks passed
@dimitrilw dimitrilw deleted the 13-bart-formula branch October 27, 2023 18:31
@dimitrilw
Copy link
Contributor Author

and just closing loop on this:
image

🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

new tap: bart / bartholomew
3 participants