Show dependencies feature #1012 by fzanollo · Pull Request #1165 · microsoft/winget-cli

fzanollo (fzanollo) · 2021-06-14T17:35:43Z

Contains:

implementation of show dependencies feature for commands: install, show, upgrade, uninstall, validate, import.
everything under a new experimental feature
unit testcases for each of the commands
Closes Show dependencies #1012

Microsoft Reviewers: Open in CodeFlow

… import

…nciesFlow

…dencyList; add Dependency to context data

github-actions · 2021-06-18T18:47:00Z

Misspellings found, please review:

deps

To accept these changes, run the following commands from this repository on this branch

pushd $(git rev-parse --show-toplevel)
perl -e '
my @expect_files=qw('".github/actions/spelling/expect.txt"');
@ARGV=@expect_files;
my @stale=qw('"bcp cdn compschema Debian google INET Langs oss REINSTALLMODE smartscreen sourc Standalone vamus Wunused "');
my $re=join "|", @stale;
my $suffix=".".time();
my $previous="";
sub maybe_unlink { unlink($_[0]) if $_[0]; }
while (<>) {
  if ($ARGV ne $old_argv) { maybe_unlink($previous); $previous="$ARGV$suffix"; rename($ARGV, $previous); open(ARGV_OUT, ">$ARGV"); select(ARGV_OUT); $old_argv = $ARGV; }
  next if /^(?:$re)(?:(?:\r|\n)*$| .*)/; print;
}; maybe_unlink($previous);'
perl -e '
my $new_expect_file=".github/actions/spelling/expect.txt";
use File::Path qw(make_path);
make_path ".github/actions/spelling";
open FILE, q{<}, $new_expect_file; chomp(my @words = <FILE>); close FILE;
my @add=qw('"debian deps inet langs "');
my %items; @items{@words} = @words x (1); @items{@add} = @add x (1);
@words = sort {lc($a) cmp lc($b)} keys %items;
open FILE, q{>}, $new_expect_file; for my $word (@words) { print FILE "$word\n" if $word =~ /\w/; };
close FILE;'
popd

…DpeendencyType

Co-authored-by: JohnMcPMS <johnmcp@microsoft.com>

…encies is now a WorkflowTask receiving resource string id

…w, move dependency related function to DependenciesFlow

github-actions · 2021-06-23T18:18:29Z

Misspellings found, please review:

begining

To accept these changes, run the following commands from this repository on this branch

pushd $(git rev-parse --show-toplevel)
perl -e '
my @expect_files=qw('".github/actions/spelling/expect.txt"');
@ARGV=@expect_files;
my @stale=qw('"bcp cdn compschema Debian google INET Langs oss REINSTALLMODE smartscreen sourc Standalone vamus Wunused "');
my $re=join "|", @stale;
my $suffix=".".time();
my $previous="";
sub maybe_unlink { unlink($_[0]) if $_[0]; }
while (<>) {
  if ($ARGV ne $old_argv) { maybe_unlink($previous); $previous="$ARGV$suffix"; rename($ARGV, $previous); open(ARGV_OUT, ">$ARGV"); select(ARGV_OUT); $old_argv = $ARGV; }
  next if /^(?:$re)(?:(?:\r|\n)*$| .*)/; print;
}; maybe_unlink($previous);'
perl -e '
my $new_expect_file=".github/actions/spelling/expect.txt";
use File::Path qw(make_path);
make_path ".github/actions/spelling";
open FILE, q{<}, $new_expect_file; chomp(my @words = <FILE>); close FILE;
my @add=qw('"begining debian inet langs "');
my %items; @items{@words} = @words x (1); @items{@add} = @add x (1);
@words = sort {lc($a) cmp lc($b)} keys %items;
open FILE, q{>}, $new_expect_file; for my $word (@words) { print FILE "$word\n" if $word =~ /\w/; };
close FILE;'
popd

github-actions · 2021-06-23T19:18:28Z

Misspellings found, please review:

begining

To accept these changes, run the following commands from this repository on this branch

pushd $(git rev-parse --show-toplevel)
perl -e '
my @expect_files=qw('".github/actions/spelling/expect.txt"');
@ARGV=@expect_files;
my @stale=qw('"bcp cdn compschema Debian google INET Langs oss REINSTALLMODE smartscreen sourc Standalone vamus Wunused "');
my $re=join "|", @stale;
my $suffix=".".time();
my $previous="";
sub maybe_unlink { unlink($_[0]) if $_[0]; }
while (<>) {
  if ($ARGV ne $old_argv) { maybe_unlink($previous); $previous="$ARGV$suffix"; rename($ARGV, $previous); open(ARGV_OUT, ">$ARGV"); select(ARGV_OUT); $old_argv = $ARGV; }
  next if /^(?:$re)(?:(?:\r|\n)*$| .*)/; print;
}; maybe_unlink($previous);'
perl -e '
my $new_expect_file=".github/actions/spelling/expect.txt";
use File::Path qw(make_path);
make_path ".github/actions/spelling";
open FILE, q{<}, $new_expect_file; chomp(my @words = <FILE>); close FILE;
my @add=qw('"begining debian inet langs "');
my %items; @items{@words} = @words x (1); @items{@add} = @add x (1);
@words = sort {lc($a) cmp lc($b)} keys %items;
open FILE, q{>}, $new_expect_file; for my $word (@words) { print FILE "$word\n" if $word =~ /\w/; };
close FILE;'
popd

…ion if needed

…installer are preferred

github-actions · 2021-06-24T17:13:27Z

Misspellings found, please review:

IISOn
Multideclaration

To accept these changes, run the following commands from this repository on this branch

pushd $(git rev-parse --show-toplevel)
perl -e '
my @expect_files=qw('".github/actions/spelling/expect.txt"');
@ARGV=@expect_files;
my @stale=qw('"bcp cdn compschema Debian google INET Langs oss REINSTALLMODE smartscreen sourc Standalone vamus Wunused "');
my $re=join "|", @stale;
my $suffix=".".time();
my $previous="";
sub maybe_unlink { unlink($_[0]) if $_[0]; }
while (<>) {
  if ($ARGV ne $old_argv) { maybe_unlink($previous); $previous="$ARGV$suffix"; rename($ARGV, $previous); open(ARGV_OUT, ">$ARGV"); select(ARGV_OUT); $old_argv = $ARGV; }
  next if /^(?:$re)(?:(?:\r|\n)*$| .*)/; print;
}; maybe_unlink($previous);'
perl -e '
my $new_expect_file=".github/actions/spelling/expect.txt";
use File::Path qw(make_path);
make_path ".github/actions/spelling";
open FILE, q{<}, $new_expect_file; chomp(my @words = <FILE>); close FILE;
my @add=qw('"debian IISOn inet langs Multideclaration "');
my %items; @items{@words} = @words x (1); @items{@add} = @add x (1);
@words = sort {lc($a) cmp lc($b)} keys %items;
open FILE, q{>}, $new_expect_file; for my $word (@words) { print FILE "$word\n" if $word =~ /\w/; };
close FILE;'
popd

github-actions · 2021-06-24T19:19:12Z

Misspellings found, please review:

standalone

To accept these changes, run the following commands from this repository on this branch

pushd $(git rev-parse --show-toplevel)
perl -e '
my @expect_files=qw('".github/actions/spelling/expect.txt"');
@ARGV=@expect_files;
my @stale=qw('"abifind CLSID combaseapi cppwinrtprocess Globals HRESULTs IApp idl IFind IGet iinstall MIDL Packageend pseudocode Xaml "');
my $re=join "|", @stale;
my $suffix=".".time();
my $previous="";
sub maybe_unlink { unlink($_[0]) if $_[0]; }
while (<>) {
  if ($ARGV ne $old_argv) { maybe_unlink($previous); $previous="$ARGV$suffix"; rename($ARGV, $previous); open(ARGV_OUT, ">$ARGV"); select(ARGV_OUT); $old_argv = $ARGV; }
  next if /^(?:$re)(?:(?:\r|\n)*$| .*)/; print;
}; maybe_unlink($previous);'
perl -e '
my $new_expect_file=".github/actions/spelling/expect.txt";
use File::Path qw(make_path);
make_path ".github/actions/spelling";
open FILE, q{<}, $new_expect_file; chomp(my @words = <FILE>); close FILE;
my @add=qw('"clsid globals hresults midl standalone "');
my %items; @items{@words} = @words x (1); @items{@add} = @add x (1);
@words = sort {lc($a) cmp lc($b)} keys %items;
open FILE, q{>}, $new_expect_file; for my $word (@words) { print FILE "$word\n" if $word =~ /\w/; };
close FILE;'
popd

github-actions · 2021-06-24T19:52:38Z

Misspellings found, please review:

standalone

To accept these changes, run the following commands from this repository on this branch

pushd $(git rev-parse --show-toplevel)
perl -e '
my @expect_files=qw('".github/actions/spelling/expect.txt"');
@ARGV=@expect_files;
my @stale=qw('"abifind CLSID combaseapi cppwinrtprocess Globals HRESULTs IApp idl IFind IGet iinstall MIDL Packageend pseudocode Xaml "');
my $re=join "|", @stale;
my $suffix=".".time();
my $previous="";
sub maybe_unlink { unlink($_[0]) if $_[0]; }
while (<>) {
  if ($ARGV ne $old_argv) { maybe_unlink($previous); $previous="$ARGV$suffix"; rename($ARGV, $previous); open(ARGV_OUT, ">$ARGV"); select(ARGV_OUT); $old_argv = $ARGV; }
  next if /^(?:$re)(?:(?:\r|\n)*$| .*)/; print;
}; maybe_unlink($previous);'
perl -e '
my $new_expect_file=".github/actions/spelling/expect.txt";
use File::Path qw(make_path);
make_path ".github/actions/spelling";
open FILE, q{<}, $new_expect_file; chomp(my @words = <FILE>); close FILE;
my @add=qw('"clsid globals hresults midl standalone "');
my %items; @items{@words} = @words x (1); @items{@add} = @add x (1);
@words = sort {lc($a) cmp lc($b)} keys %items;
open FILE, q{>}, $new_expect_file; for my $word (@words) { print FILE "$word\n" if $word =~ /\w/; };
close FILE;'
popd

github-actions · 2021-06-24T21:06:18Z

Misspellings found, please review:

standalone

To accept these changes, run the following commands from this repository on this branch

pushd $(git rev-parse --show-toplevel)
perl -e '
my @expect_files=qw('".github/actions/spelling/expect.txt"');
@ARGV=@expect_files;
my @stale=qw('"abifind CLSID combaseapi cppwinrtprocess Globals HRESULTs IApp idl IFind IGet iinstall MIDL Packageend pseudocode Xaml "');
my $re=join "|", @stale;
my $suffix=".".time();
my $previous="";
sub maybe_unlink { unlink($_[0]) if $_[0]; }
while (<>) {
  if ($ARGV ne $old_argv) { maybe_unlink($previous); $previous="$ARGV$suffix"; rename($ARGV, $previous); open(ARGV_OUT, ">$ARGV"); select(ARGV_OUT); $old_argv = $ARGV; }
  next if /^(?:$re)(?:(?:\r|\n)*$| .*)/; print;
}; maybe_unlink($previous);'
perl -e '
my $new_expect_file=".github/actions/spelling/expect.txt";
use File::Path qw(make_path);
make_path ".github/actions/spelling";
open FILE, q{<}, $new_expect_file; chomp(my @words = <FILE>); close FILE;
my @add=qw('"clsid globals hresults midl standalone "');
my %items; @items{@words} = @words x (1); @items{@add} = @add x (1);
@words = sort {lc($a) cmp lc($b)} keys %items;
open FILE, q{>}, $new_expect_file; for my $word (@words) { print FILE "$word\n" if $word =~ /\w/; };
close FILE;'
popd

…lation

fzanollo (fzanollo) added 14 commits June 4, 2021 18:16

first tests for show depenencies feature

9c72b02

show dependencies feature for show, install, need to change message

0a81ec5

showflow output changed

b416985

install commands shows dependencies as in specs

c848824

tests for informing dependencies on commands: show, install, upgrade,…

688cbe5

… import

refactor install flow

7b488f1

change import command from install flow (install multiple)

c8d02ea

import shows all dependencies together

72fd209

show dependencies for validate and uninstall

d6d0c53

tests for validate and uninstall

d3063f2

create show dependencies exp feature

9d05f6f

test for validate command

e3ccc8e

put functionality under experimental feature check

44fe9e2

enable show dep experimental feature on unit test cases

a69b930

fzanollo (fzanollo) requested a review from JohnMcPMS June 14, 2021 17:35

fzanollo (fzanollo) requested a review from a team as a code owner June 14, 2021 17:35

ghost added the Issue-Feature This is a feature request for the Windows Package Manager client. label Jun 14, 2021

JohnMcPMS requested changes Jun 15, 2021

View reviewed changes

ghost added the Needs-Author-Feedback Issue needs attention from issue or PR author label Jun 15, 2021

change experimental feature name

8bb3aa1

ghost removed the Needs-Author-Feedback Issue needs attention from issue or PR author label Jun 15, 2021

fzanollo (fzanollo) added 5 commits June 15, 2021 14:04

remove info stream characters on testcase

9737ae1

move check for exp feature inside report function, create new Depende…

49c7fea

…nciesFlow

DependenciesFlow header and cpp inside Workflow

74d29ff

�change representation of Dependency, create DependencyType and Depen…

b7590c3

…dencyList; add Dependency to context data

dependencies context data is of type DependencyList (not optional)

b04225a

fzanollo (fzanollo) added 2 commits June 18, 2021 18:54

fix spelling errors

2a620d8

ApplyTo function, can receibe a lambda function to apply on specific …

40e05b4

…DpeendencyType

JohnMcPMS requested changes Jun 21, 2021

View reviewed changes

JohnMcPMS requested changes Jun 22, 2021

View reviewed changes

ghost added Needs-Author-Feedback Issue needs attention from issue or PR author and removed Needs-Author-Feedback Issue needs attention from issue or PR author labels Jun 22, 2021

fzanollo (fzanollo) and others added 4 commits June 23, 2021 13:04

Update src/AppInstallerCLIPackage/Shared/Strings/en-us/winget.resw

c785b14

Co-authored-by: JohnMcPMS <johnmcp@microsoft.com>

compare id with ICU case insensitive

0a31f79

move dependencies related functions to DependenciesFlow, ReportDepend…

af7b09c

…encies is now a WorkflowTask receiving resource string id

TODO to get dependencies on uninstall context, remove testcase for no…

8fce81d

…w, move dependency related function to DependenciesFlow

wfIds not temporary

af036e4

fzanollo (fzanollo) added 2 commits June 23, 2021 18:17

DependencyList Add function checks for existence and updates min vers…

3388ea7

…ion if needed

root dependencies are used when installer are not present, otherwise …

f07fdc6

…installer are preferred

fzanollo (fzanollo) added 2 commits June 24, 2021 15:32

add tests names to spelling expect

5eb6270

merge commits from master

c8e9e27

JohnMcPMS reviewed Jun 24, 2021

View reviewed changes

Comment thread src/AppInstallerCommonCore/Public/winget/ManifestCommon.h Outdated

Comment thread src/AppInstallerCommonCore/Public/winget/ManifestCommon.h

Comment thread src/AppInstallerCommonCore/Public/winget/ManifestCommon.h Outdated

style and details

0bbe093

fix merge conflicts

f0f3676

spellcheck exception

dd03b85

fzanollo (fzanollo) requested a review from JohnMcPMS June 25, 2021 13:16

JohnMcPMS reviewed Jun 25, 2021

View reviewed changes

Comment thread src/AppInstallerCommonCore/Public/winget/ManifestCommon.h Outdated

fzanollo (fzanollo) added 2 commits June 28, 2021 12:31

detail when adding dep

2985cd5

divide identity report and installation disclaimer from actual instal…

4e623d4

…lation

fzanollo (fzanollo) requested a review from JohnMcPMS July 19, 2021 14:18

JohnMcPMS approved these changes Jul 29, 2021

View reviewed changes

merge with master

8eede60

JohnMcPMS merged commit fe8ac77 into microsoft:master Aug 3, 2021

Uh oh!

Conversation

fzanollo (fzanollo) commented Jun 14, 2021 • edited by ghost Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Microsoft Reviewers: Open in CodeFlow

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 18, 2021

Misspellings found, please review:

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 23, 2021

Misspellings found, please review:

Uh oh!

github-actions Bot commented Jun 23, 2021

Misspellings found, please review:

Uh oh!

github-actions Bot commented Jun 24, 2021

Misspellings found, please review:

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 24, 2021

Misspellings found, please review:

Uh oh!

github-actions Bot commented Jun 24, 2021

Misspellings found, please review:

Uh oh!

github-actions Bot commented Jun 24, 2021

Misspellings found, please review:

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

fzanollo (fzanollo) commented Jun 14, 2021 •

edited by ghost

Loading