Skip to content

[dotnet] include snupkg files when packaging things up and allow the use of sourcelink#17467

Merged
AutomatedTester merged 3 commits into
trunkfrom
busy-bassi-b3be33
May 18, 2026
Merged

[dotnet] include snupkg files when packaging things up and allow the use of sourcelink#17467
AutomatedTester merged 3 commits into
trunkfrom
busy-bassi-b3be33

Conversation

@AutomatedTester
Copy link
Copy Markdown
Member

@AutomatedTester AutomatedTester commented May 15, 2026

🔗 Related Issues

💥 What does this PR do?

🔧 Implementation Notes

🤖 AI assistance

  • No substantial AI assistance used
  • AI assisted (complete below)
    • Tool(s):
    • What was generated:
    • I reviewed all AI output and can explain the change

💡 Additional Considerations

🔄 Types of changes

  • Cleanup (formatting, renaming)
  • Bug fix (backwards compatible)
  • New feature (non-breaking change which adds functionality and tests!)
  • Breaking change (fix or feature that would cause existing functionality to change)

@selenium-ci selenium-ci added C-dotnet .NET Bindings B-build Includes scripting, bazel and CI integrations labels May 15, 2026
@AutomatedTester AutomatedTester changed the title [dotnet] include snupkg files when packaging things up [dotnet] include snupkg files when packaging things up and allow the use of sourcelink May 15, 2026
@AutomatedTester AutomatedTester mentioned this pull request May 15, 2026
Closed
3 tasks
shs96c
shs96c approved these changes May 18, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@shs96c shs96c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment, but LGTM

Comment thread dotnet/private/nuget_push.bzl
@AutomatedTester AutomatedTester merged commit 79ab3f4 into trunk May 18, 2026
30 checks passed
@AutomatedTester AutomatedTester deleted the busy-bassi-b3be33 branch May 18, 2026 08:46
@nvborisenko nvborisenko mentioned this pull request May 18, 2026
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shs96c shs96c shs96c approved these changes

Assignees

No one assigned

Labels

B-build Includes scripting, bazel and CI integrations C-dotnet .NET Bindings

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@AutomatedTester @shs96c @selenium-ci