-
Notifications
You must be signed in to change notification settings - Fork 225
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
Build: Bump lando/notarize-action from 4f5869b09386e8336802159031e4189e0919ae20 to bd2f055b8685623053d14594e9c5742c912befaf #3263
Conversation
Needs review. |
@dependabot rebase |
Bumps [lando/notarize-action](https://github.com/lando/notarize-action) from 4f5869b09386e8336802159031e4189e0919ae20 to bd2f055b8685623053d14594e9c5742c912befaf. - [Release notes](https://github.com/lando/notarize-action/releases) - [Changelog](https://github.com/lando/notarize-action/blob/main/CHANGELOG.md) - [Commits](lando/notarize-action@4f5869b...bd2f055) --- updated-dependencies: - dependency-name: lando/notarize-action dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]>
f6775a2
to
973b8ac
Compare
What kind of review? I've looked at the code for the action at https://github.com/lando/notarize-action/commits/main/, although I'm not familiar with the project. We can verify that our project builds correctly with this PR, but I guess the operation of notarize-action can't be tested unless we are signing a release? Does signing/notarization currently work in our repo? I would be inclined to merge this, and investigate any problems if they occur later. |
I don't think we notarize ourselves. So I don't think we can test this. |
In my opinion it's an avoidable dependency. I'm a bit worried about a security related action we don't fully understand. |
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.
Probably we need to go with this.
@dependabot close |
Bumps lando/notarize-action from 4f5869b09386e8336802159031e4189e0919ae20 to bd2f055b8685623053d14594e9c5742c912befaf.
Changelog
Sourced from lando/notarize-action's changelog.
Commits
bd2f055
Contrib plus (#28)b1d2a24
upraYou can trigger a rebase of this PR by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)