fix: use AfterTargets="Restore" to support NuGet credential helpers for private feeds

docs/guide/automate.md

@@ -25,7 +25,7 @@ You can set the `HUSKY` environment variable to `0` in order to disable husky in
 To manually attach husky to your project, add the below code to one of your projects (*.csproj/*.vbproj).
 
 ``` xml:no-line-numbers:no-v-pre
-<Target Name="husky" BeforeTargets="Restore;CollectPackageReferences" Condition="'$(HUSKY)' != 0"
+<Target Name="husky" AfterTargets="Restore" Condition="'$(HUSKY)' != 0"
         Inputs="../../.config/dotnet-tools.json"
         Outputs="../../.husky/_/install.stamp">
    <Exec Command="dotnet tool restore"  StandardOutputImportance="Low" StandardErrorImportance="High"/>
@@ -59,7 +59,7 @@ to avoid this, you can add the `$(IsCrossTargetingBuild)' == 'true'` condition t
 e.g
 
 ``` xml:no-line-numbers:no-v-pre
-<Target Name="husky" BeforeTargets="Restore;CollectPackageReferences" Condition="'$(HUSKY)' != 0 and '$(IsCrossTargetingBuild)' == 'true'">
+<Target Name="husky" AfterTargets="Restore" Condition="'$(HUSKY)' != 0 and '$(IsCrossTargetingBuild)' == 'true'">
 ...
 ```
 

docs/guide/submodules.md

@@ -34,7 +34,7 @@ The `attach` command offers a `--ignore-submodule` options that generates an MsB
 The generated block will look something like this, If you're attaching husky manually copy the target to your `.csproj` and adjust `WorkingDirectory` accordingly.
 
 ```xml:no-line-numbers:no-v-pre
-<Target Name="husky" BeforeTargets="Restore;CollectPackageReferences" Condition="'$(HUSKY)' != 0  and '$(IgnoreSubmodule)' != 0">
+<Target Name="husky" AfterTargets="Restore" Condition="'$(HUSKY)' != 0  and '$(IgnoreSubmodule)' != 0">
    <Exec Command="dotnet tool restore"  StandardOutputImportance="Low" StandardErrorImportance="High"/>
    <Exec Command="dotnet husky install --ignore-submodule" StandardOutputImportance="Low" StandardErrorImportance="High"
          WorkingDirectory="../../" />  <!--Update this to the relative path to your project root dir -->

src/Husky/Cli/AttachCommand.cs

@@ -67,7 +67,7 @@ private XElement GetTarget(string condition, string rootRelativePath)
 
       var target = new XElement("Target");
       target.SetAttributeValue("Name", "Husky");
-      target.SetAttributeValue("BeforeTargets", "Restore;CollectPackageReferences");
+      target.SetAttributeValue("AfterTargets", "Restore");
       target.SetAttributeValue("Condition", condition);
       target.SetAttributeValue("Inputs", inputPath);
       target.SetAttributeValue("Outputs", sentinelPath);

tests/HuskyTest/Cli/AttachCommandTests.cs

@@ -66,6 +66,9 @@
       huskyTarget.Descendants("ItemGroup").Descendants("FileWrites").Should().HaveCount(1);
 
       _console.ReadOutputString().Trim().Should().Be("Husky dev-dependency successfully attached to this project.");
+
+      huskyTarget.Attribute("AfterTargets")?.Value.Should().Be("Restore");
+      huskyTarget.Attribute("BeforeTargets").Should().BeNull();
    }
 
    [Fact]
@@ -145,7 +148,7 @@
          .SingleOrDefault(q => q.Attribute("Name")?.Value == "Husky")?.Descendants("Exec");
 
       targetExecElements.Should().NotBeNull().And.HaveCount(2);
       targetExecElements.SingleOrDefault(e => e.Attribute("Command").Value.Contains("dotnet husky install --ignore-submodule")).Should().NotBeNull();
       _xmlIo.Received(1).Save(Arg.Any<string>(), Arg.Any<XElement>());
    }