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

Add --update-suite to sforceimport to allow updates without user interaction #277

Closed
wants to merge 1 commit into from

Conversation

smarr
Copy link
Collaborator

@smarr smarr commented Aug 1, 2023

I'd like to be able to use sforceimport to update my suite.py in a cronjob, without user interaction.

I see two options:

  1. add a new flag like this PR, --update-suite or similar to sforceimport
  2. or add a default value for ask_yes_no, though, this option changes the current behavior minimally, which might be more risky. Don't know.

This is useful for scripts since ask_yes_no checks for an interactive session and bails out without one, if there's no default value.

…raction

This is useful for scripts since ask_yes_no checks for an interactive session and bails out without one.
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 1, 2023
@smarr
Copy link
Collaborator Author

smarr commented Aug 1, 2023

Hm, the gate is failing. It looks like jq is trying to find an pip key in common.json, which has been failing for a while.

@eregon
Copy link
Member

eregon commented Aug 1, 2023

Isn't mx's option -y answer 'y' to all questions asked enough?

@smarr
Copy link
Collaborator Author

smarr commented Aug 1, 2023

Yepp, I overlooked that. Thanks!

@smarr smarr closed this Aug 1, 2023
@smarr smarr deleted the sforceimport-update-suite branch August 1, 2023 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants