Skip to content

Conversation

@edumserrano
Copy link
Owner

@edumserrano edumserrano commented Aug 12, 2023

Before this PR I was delegating the generation of release notes to the ncipollo/release-action. However I soon found myself requiring to specify the previous tag when auto generating release notes which this action did not support.

In my case the problem is that I publish two NuGet packages from this repo and for the auto generated release notes to be accurate I need them to be generated between the last release tags for each NuGet package release, not just for whatever is the latest tag.

For instance, if I have the following releases:

  • nuget-a-v3
  • nuget-b-v2
  • nuget-a-v2

And now I'm going to do a release with for nuget-b, let's say nuget-b-v3 then I want the release notes to be generated by comparing the tag nuget-b-v2 and nuget-b-v3. By default that's not what was happening before this PR, it would grab the latest tab and would generated release notes between the tag nuget-a-v3 and nuget-b-v3 which didn't make sense.

This PR takes the suggestion from ncipollo/release-action#198 and calls the generate release notes GitHub endpoint and then passes the resulting output to the ncipollo/release-action.

Note

  1. Temporarily disabled publishing to nuget.org to test this out.
  2. Any commits that are needed to fix minor issues with this workflow change as well as the commit to re-enable the publishing to nuget.org, will be associated with this PR.

@github-actions
Copy link

Test runs on windows-latest for commit 2e8931d

✔️ Pass - DotNet.Sdk.Extensions.Testing.Tests.dll on .NETCoreApp,Version=v6.0 ---- ## Run Summary

Overall Result: ✔️ Pass
Pass Rate: 100%
Total Tests: 149

Date: 2023-08-12 17:42:47 - 2023-08-12 17:42:51
Run Duration: 3s 910ms

GitHub Runner OS: windows-latest
Operating System: Microsoft Windows 10.0.20348
Framework: .NETCoreApp,Version=v6.0
Assembly: DotNet.Sdk.Extensions.Testing.Tests.dll

✔️ Passed ❌ Failed ⚠️ Skipped
149 0 0
100% 0% 0%
## Run Messages
Informational
 [xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.5.0.1+5ebf84cd75 (64-bit .NET 6.0.21) [xUnit.net 00:00:00.13]   Discovering: DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:00.45]   Discovered:  DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:00.45]   Starting:    DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:03.10]   Finished:    DotNet.Sdk.Extensions.Testing.Tests 
----
✔️ Pass - DotNet.Sdk.Extensions.Tests.dll on .NETCoreApp,Version=v6.0 ---- ## Run Summary

Overall Result: ✔️ Pass
Pass Rate: 100%
Total Tests: 157

Date: 2023-08-12 17:42:52 - 2023-08-12 17:43:04
Run Duration: 11s 436ms

GitHub Runner OS: windows-latest
Operating System: Microsoft Windows 10.0.20348
Framework: .NETCoreApp,Version=v6.0
Assembly: DotNet.Sdk.Extensions.Tests.dll

✔️ Passed ❌ Failed ⚠️ Skipped
157 0 0
100% 0% 0%
## Run Messages
Informational
 [xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.5.0.1+5ebf84cd75 (64-bit .NET 6.0.21) [xUnit.net 00:00:00.12]   Discovering: DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:00.23]   Discovered:  DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:00.23]   Starting:    DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:10.79]   Finished:    DotNet.Sdk.Extensions.Tests 
----
✔️ Pass - DotNet.Sdk.Extensions.Testing.Tests.dll on .NETCoreApp,Version=v7.0 ---- ## Run Summary

Overall Result: ✔️ Pass
Pass Rate: 100%
Total Tests: 149

Date: 2023-08-12 17:43:07 - 2023-08-12 17:43:09
Run Duration: 2s 193ms

GitHub Runner OS: windows-latest
Operating System: Microsoft Windows 10.0.20348
Framework: .NETCoreApp,Version=v7.0
Assembly: DotNet.Sdk.Extensions.Testing.Tests.dll

✔️ Passed ❌ Failed ⚠️ Skipped
149 0 0
100% 0% 0%
## Run Messages
Informational
 [xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.5.0.1+5ebf84cd75 (64-bit .NET 7.0.10) [xUnit.net 00:00:00.10]   Discovering: DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:00.27]   Discovered:  DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:00.27]   Starting:    DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:01.62]   Finished:    DotNet.Sdk.Extensions.Testing.Tests 
----
✔️ Pass - DotNet.Sdk.Extensions.Tests.dll on .NETCoreApp,Version=v7.0 ---- ## Run Summary

Overall Result: ✔️ Pass
Pass Rate: 100%
Total Tests: 157

Date: 2023-08-12 17:43:10 - 2023-08-12 17:43:21
Run Duration: 11s 8ms

GitHub Runner OS: windows-latest
Operating System: Microsoft Windows 10.0.20348
Framework: .NETCoreApp,Version=v7.0
Assembly: DotNet.Sdk.Extensions.Tests.dll

✔️ Passed ❌ Failed ⚠️ Skipped
157 0 0
100% 0% 0%
## Run Messages
Informational
 [xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.5.0.1+5ebf84cd75 (64-bit .NET 7.0.10) [xUnit.net 00:00:00.11]   Discovering: DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:00.20]   Discovered:  DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:00.20]   Starting:    DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:10.42]   Finished:    DotNet.Sdk.Extensions.Tests 
----

@github-actions
Copy link

Test runs on ubuntu-latest for commit 2e8931d

✔️ Pass - DotNet.Sdk.Extensions.Testing.Tests.dll on .NETCoreApp,Version=v6.0 ---- ## Run Summary

Overall Result: ✔️ Pass
Pass Rate: 100%
Total Tests: 149

Date: 2023-08-12 17:40:45 - 2023-08-12 17:40:49
Run Duration: 3s 826ms

GitHub Runner OS: ubuntu-latest
Operating System: Linux 5.15.0-1042-azure #49-Ubuntu SMP Tue Jul 11 17:28:46 UTC 2023
Framework: .NETCoreApp,Version=v6.0
Assembly: DotNet.Sdk.Extensions.Testing.Tests.dll

✔️ Passed ❌ Failed ⚠️ Skipped
149 0 0
100% 0% 0%
## Run Messages
Informational
 [xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.5.0.1+5ebf84cd75 (64-bit .NET 6.0.21) [xUnit.net 00:00:00.13]   Discovering: DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:00.39]   Discovered:  DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:00.39]   Starting:    DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:03.02]   Finished:    DotNet.Sdk.Extensions.Testing.Tests 
----
✔️ Pass - DotNet.Sdk.Extensions.Tests.dll on .NETCoreApp,Version=v6.0 ---- ## Run Summary

Overall Result: ✔️ Pass
Pass Rate: 100%
Total Tests: 157

Date: 2023-08-12 17:40:50 - 2023-08-12 17:41:01
Run Duration: 11s 351ms

GitHub Runner OS: ubuntu-latest
Operating System: Linux 5.15.0-1042-azure #49-Ubuntu SMP Tue Jul 11 17:28:46 UTC 2023
Framework: .NETCoreApp,Version=v6.0
Assembly: DotNet.Sdk.Extensions.Tests.dll

✔️ Passed ❌ Failed ⚠️ Skipped
157 0 0
100% 0% 0%
## Run Messages
Informational
 [xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.5.0.1+5ebf84cd75 (64-bit .NET 6.0.21) [xUnit.net 00:00:00.14]   Discovering: DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:00.27]   Discovered:  DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:00.28]   Starting:    DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:10.54]   Finished:    DotNet.Sdk.Extensions.Tests 
----
✔️ Pass - DotNet.Sdk.Extensions.Testing.Tests.dll on .NETCoreApp,Version=v7.0 ---- ## Run Summary

Overall Result: ✔️ Pass
Pass Rate: 100%
Total Tests: 149

Date: 2023-08-12 17:41:05 - 2023-08-12 17:41:09
Run Duration: 4s 168ms

GitHub Runner OS: ubuntu-latest
Operating System: Linux 5.15.0-1042-azure #49-Ubuntu SMP Tue Jul 11 17:28:46 UTC 2023
Framework: .NETCoreApp,Version=v7.0
Assembly: DotNet.Sdk.Extensions.Testing.Tests.dll

✔️ Passed ❌ Failed ⚠️ Skipped
149 0 0
100% 0% 0%
## Run Messages
Informational
 [xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.5.0.1+5ebf84cd75 (64-bit .NET 7.0.10) [xUnit.net 00:00:00.12]   Discovering: DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:00.35]   Discovered:  DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:00.36]   Starting:    DotNet.Sdk.Extensions.Testing.Tests [xUnit.net 00:00:03.43]   Finished:    DotNet.Sdk.Extensions.Testing.Tests 
----
✔️ Pass - DotNet.Sdk.Extensions.Tests.dll on .NETCoreApp,Version=v7.0 ---- ## Run Summary

Overall Result: ✔️ Pass
Pass Rate: 100%
Total Tests: 157

Date: 2023-08-12 17:41:10 - 2023-08-12 17:41:21
Run Duration: 11s 210ms

GitHub Runner OS: ubuntu-latest
Operating System: Linux 5.15.0-1042-azure #49-Ubuntu SMP Tue Jul 11 17:28:46 UTC 2023
Framework: .NETCoreApp,Version=v7.0
Assembly: DotNet.Sdk.Extensions.Tests.dll

✔️ Passed ❌ Failed ⚠️ Skipped
157 0 0
100% 0% 0%
## Run Messages
Informational
 [xUnit.net 00:00:00.00] xUnit.net VSTest Adapter v2.5.0.1+5ebf84cd75 (64-bit .NET 7.0.10) [xUnit.net 00:00:00.15]   Discovering: DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:00.41]   Discovered:  DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:00.42]   Starting:    DotNet.Sdk.Extensions.Tests [xUnit.net 00:00:10.40]   Finished:    DotNet.Sdk.Extensions.Tests 
----

@codecov
Copy link

codecov bot commented Aug 12, 2023

Codecov Report

Merging #734 (2e8931d) into main (bfe4b75) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #734   +/-   ##
=======================================
  Coverage   86.72%   86.72%           
=======================================
  Files          77       77           
  Lines        1409     1409           
  Branches      219      219           
=======================================
  Hits         1222     1222           
  Misses        100      100           
  Partials       87       87           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@edumserrano edumserrano merged commit b164b6d into main Aug 12, 2023
@edumserrano edumserrano deleted the release-notes branch August 12, 2023 17:46
edumserrano added a commit that referenced this pull request Aug 12, 2023
edumserrano added a commit that referenced this pull request Aug 12, 2023
edumserrano added a commit that referenced this pull request Aug 13, 2023
edumserrano added a commit that referenced this pull request Aug 13, 2023
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