Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

refactor(scoop-info): Use List View for output #4741

Merged
merged 2 commits into from
Feb 17, 2022
Merged

Conversation

rashil2000
Copy link
Member

Description

scoop-info was already using Write-Output. This PR just wraps that output in a PSCustomObject.

Motivation and Context

Closes #4475

How Has This Been Tested?

Before

image

After

image

Pipeline

image

Checklist:

  • I have read the Contributing Guide.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.

@rashil2000
Copy link
Member Author

I will add lots of new details to scoop-info - last updated date, remote URL, dependencies, suggestions, support for local filepaths - so this PR is essential to be merged before.

libexec/scoop-info.ps1 Outdated Show resolved Hide resolved
libexec/scoop-info.ps1 Outdated Show resolved Hide resolved
@rashil2000
Copy link
Member Author

Done.

@rashil2000 rashil2000 merged commit 2651764 into develop Feb 17, 2022
@rashil2000 rashil2000 deleted the refactor-info branch February 17, 2022 02:51
se35710 pushed a commit to se35710/scoop that referenced this pull request Mar 8, 2022
* refactor(scoop-info): Use List View for output

* review comments
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants