Skip to content

Fix bug in the setup of a profile through verdi setup/quicksetup#2395

Merged
giovannipizzi merged 1 commit into
aiidateam:provenance_redesignfrom
sphuber:fix_2394_verdi_setup_keep_existing_profile
Jan 17, 2019
Merged

Fix bug in the setup of a profile through verdi setup/quicksetup#2395
giovannipizzi merged 1 commit into
aiidateam:provenance_redesignfrom
sphuber:fix_2394_verdi_setup_keep_existing_profile

Conversation

@sphuber
Copy link
Copy Markdown
Contributor

@sphuber sphuber commented Jan 16, 2019

Fixes #2394

When setting up a profile that already exists, verdi is supposed
prompt the user whether they want to reuse the information of the
existing profile or if they want to change it. However, instead of
asking to confirm, the command was prompting, causing the user to always
have to reenter the information.

When setting up a profile that already exists, `verdi` is supposed
prompt the user whether they want to reuse the information of the
existing profile or if they want to change it. However, instead of
asking to confirm, the command was prompting, causing the user to always
have to reenter the information.
@sphuber sphuber requested a review from szoupanos January 16, 2019 17:41
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-6.03%) to 62.686% when pulling e9fd073 on sphuber:fix_2394_verdi_setup_keep_existing_profile into 487c112 on aiidateam:provenance_redesign.

@coveralls
Copy link
Copy Markdown

coveralls commented Jan 16, 2019

Coverage Status

Coverage decreased (-0.003%) to 68.715% when pulling e9fd073 on sphuber:fix_2394_verdi_setup_keep_existing_profile into 487c112 on aiidateam:provenance_redesign.

@sphuber sphuber requested a review from giovannipizzi January 17, 2019 09:35
@giovannipizzi giovannipizzi merged commit de14357 into aiidateam:provenance_redesign Jan 17, 2019
@sphuber sphuber deleted the fix_2394_verdi_setup_keep_existing_profile branch January 17, 2019 12:25
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.

3 participants