Skip to content
This repository was archived by the owner on Oct 31, 2023. It is now read-only.

Building on top of #1#2

Merged
mitchellh merged 3 commits intomasterfrom
etter-tanium-standalone-notarization
Nov 5, 2019
Merged

Building on top of #1#2
mitchellh merged 3 commits intomasterfrom
etter-tanium-standalone-notarization

Conversation

@mitchellh
Copy link
Owner

See #1

@etter-tanium - Here is what I noted in my comment. I added commits to your branch so you retain original authorship there.

etter-tanium and others added 3 commits November 4, 2019 16:33
- Add new optional notarize section to the config
- Allow specifying notarize in lieu of sources
@mitchellh mitchellh merged commit b86987e into master Nov 5, 2019
@mitchellh mitchellh deleted the etter-tanium-standalone-notarization branch November 5, 2019 05:22
@etter-tanium
Copy link
Contributor

Thanks for pulling in these changes with your improvements. I may put up another PR for the Entitlements enhancement. (Let me know if you'd prefer I use GitHub Issues for enhancement-related things, or if "drive-by PRs" are OK)
And I may have some old code to dig up for adding spit-n-polish to DMG image builds.

@mitchellh
Copy link
Owner Author

Awesome.

It’d be helpful if you just opened an issue to warn me its coming so I don’t overlap with you. That cool? If you want to include some thoughts on how you’re going to do the config we can bikeshed on that there so when the PR comes in hopefully can just merge quickly.

I appreciate the contribution!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants