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.

Sign up for GitHub

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

Jump to bottom

Feature wish: Add a --yes flag to automatically accept Confirm questions #132

Closed
erikmd opened this issue Oct 5, 2021 · 2 comments · Fixed by #158
Closed

Feature wish: Add a --yes flag to automatically accept Confirm questions #132

erikmd opened this issue Oct 5, 2021 · 2 comments · Fixed by #158

Comments

@erikmd
Copy link

erikmd commented Oct 5, 2021

In order to be able to use opam publish in "batch mode", directly from a GH Action, I believe it would be very handy to have a -y or so flag, a bit like opam 🙂

Note: I tried yes | opam publish --no-browser --msg-file=CHANGES.md --tag $tag_name $user/$repo but got:

You will be shown the patch before submitting.
Please confirm the above data. Continue ?  [Y/n] n
yes: standard output: Broken pipe
Error: Process completed with exit code 10.
erikmd added a commit to pfitaxel/learn-ocaml that referenced this issue Oct 5, 2021
@erikmd @yurug
feat(release.yml): Add a (3 jobs)-based GHA using release-please
059aea9 
* Use var OPAM_RELEASE (GitHub PAC)

* Use "expect" to workaround the lack of this feature:
  ocaml-opam/opam-publish#132

Co-Authored-By: Yann Régis-Gianas <[email protected]>
Co-Authored-By: Erik Martin-Dorel <[email protected]>
@erikmd erikmd mentioned this issue Oct 5, 2021
Merged
erikmd added a commit to pfitaxel/learn-ocaml that referenced this issue Oct 5, 2021
@erikmd @yurug
feat(release.yml): Add a (3 jobs)-based GHA using release-please
e4556e7 
* Use var OPAM_RELEASE (GitHub PAC)

* Use "expect" to workaround the lack of this feature:
  ocaml-opam/opam-publish#132

Co-Authored-By: Yann Régis-Gianas <[email protected]>
Co-Authored-By: Erik Martin-Dorel <[email protected]>
erikmd added a commit to ocaml-sf/learn-ocaml that referenced this issue Oct 5, 2021
@erikmd @yurug
feat(release.yml): Add a (3 jobs)-based GHA using release-please (#434)
7e389ef 
* Use var `OPAM_RELEASE` (GitHub PAC)

* Use `expect` to workaround the fact that the feature wish
  ocaml-opam/opam-publish#132
  is not yet available.

Co-Authored-By: Yann Régis-Gianas <[email protected]>
Co-Authored-By: Erik Martin-Dorel <[email protected]>
@phated phated mentioned this issue Nov 7, 2021
Closed
@phated
Copy link
Contributor

phated commented Nov 8, 2021

It seems that v2.1.0 of opam-publish doesn't respect OPAMYES=1 for some reason. I can't quite understand why.

@phated
Copy link
Contributor

phated commented Nov 8, 2021

I downgraded to opam-publish v2.0.3 in my local env and it works again.

@erikmd erikmd mentioned this issue Sep 6, 2023
Open
@AltGr AltGr mentioned this issue Oct 19, 2023
Merged
@kit-ty-kate kit-ty-kate linked a pull request Oct 23, 2023 that will close this issue
Add a --no-confirmation option #158
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add a --no-confirmation option ocaml-opam/opam-publish
2 participants
@phated @erikmd