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

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
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