Skip to content

feat(chore): Add missing -a/--all param to all commands#5004

Merged
rashil2000 merged 7 commits intodevelopfrom
reset-all
Jun 22, 2022
Merged

feat(chore): Add missing -a/--all param to all commands#5004
rashil2000 merged 7 commits intodevelopfrom
reset-all

Conversation

@rashil2000
Copy link
Copy Markdown
Member

@rashil2000 rashil2000 commented Jun 21, 2022

Closes #5027

Checklist:

  • I have read the Contributing Guide.
  • I have ensured that I am targeting the develop branch.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.
  • I have added an entry in the CHANGELOG.

@niheaven
Copy link
Copy Markdown
Member

LGTM for 'cache' and 'reset', but for 'virustotal', the -e sounds strange, and why not remove arch and let -a stands for --all?

Users check not-installed app's URLs for both 32bit and 64bit, right?

@rashil2000
Copy link
Copy Markdown
Member Author

Users check not-installed app's URLs for both 32bit and 64bit, right?

I'm not really sure about this.

For consistency I can remove the arch flag, and use the system's default architecture only. What should we do?

@niheaven
Copy link
Copy Markdown
Member

For consistency I can remove the arch flag, and use the system's default architecture only. What should we do?

I prefer to check both for not installed apps, and only check used architecture if installed.

niheaven
niheaven previously approved these changes Jun 22, 2022
Copy link
Copy Markdown
Member

@niheaven niheaven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rashil2000 rashil2000 merged commit 9811a5f into develop Jun 22, 2022
@rashil2000 rashil2000 deleted the reset-all branch June 22, 2022 10:17
slaughtering pushed a commit to slaughtering/scoop that referenced this pull request Jul 7, 2022
…er#5004)

* feat(scoop-reset): Add -a/--all switch to reset all apps

* feat(scoop-cache): Add -a/--all switch to delete whole cache

* feat(scoop-virustotal): Add -e/--every switch to check every installed app

* Update CHANGELOG.md

* use 'all' instead of 'every'
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.

2 participants