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

Answer to "Add user to config file?" in vt_use_report is disregarded #221

Open
jan-imbi opened this issue Dec 7, 2021 · 1 comment
Open
Assignees
Labels

Comments

@jan-imbi
Copy link

jan-imbi commented Dec 7, 2021

Answering n to the prompt "Add user to config file?" when using vt_use_report leads to a user being created anyways.

@thebioengineer
Copy link
Collaborator

That had been a conscious choice when we wrote the function to explicitly add a user to record who generated the validation report. However, I do see how the prompting (which is not done by vt_use_report() but a separate function) is misleading.

We can definitely update that. However, I would point out that the default report does require there to be users documented in the config file when it generates the table of people involved in the validation effort.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants