Skip to content

Removes snupkg artifact upload#22

Merged
st0o0 merged 1 commit intomainfrom
fix/remove-snupkg
Jan 28, 2026
Merged

Removes snupkg artifact upload#22
st0o0 merged 1 commit intomainfrom
fix/remove-snupkg

Conversation

@st0o0
Copy link
Copy Markdown
Owner

@st0o0 st0o0 commented Jan 28, 2026

Removes the generation and uploading of symbol packages (.snupkg) during the build and release process.

These packages are no longer required, simplifying the release pipeline.

Removes the generation and uploading of symbol packages (.snupkg) during the build and release process.

These packages are no longer required, simplifying the release pipeline.
@st0o0 st0o0 requested a review from LineOfC0d3 January 28, 2026 06:08
@st0o0 st0o0 self-assigned this Jan 28, 2026
@st0o0 st0o0 added the enhancement New feature label Jan 28, 2026
Copy link
Copy Markdown
Collaborator

@LineOfC0d3 LineOfC0d3 left a comment

Choose a reason for hiding this comment

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

Why remove it? Wasn't it just recently introduced in PR #18?

@st0o0
Copy link
Copy Markdown
Owner Author

st0o0 commented Jan 28, 2026

Why remove it? Wasn't it just recently introduced in PR #18?

I took a look at nupkg with Nuget.Info, and it contains all the information needed for SourceLink. Therefore, snupkg is unnecessary and not needed.

@LineOfC0d3
Copy link
Copy Markdown
Collaborator

Why remove it? Wasn't it just recently introduced in PR #18?

I took a look at nupkg with Nuget.Info, and it contains all the information needed for SourceLink. Therefore, snupkg is unnecessary and not needed.

Ok

@LineOfC0d3 LineOfC0d3 self-requested a review January 28, 2026 07:10
@st0o0 st0o0 merged commit ca2f29d into main Jan 28, 2026
5 checks passed
@st0o0 st0o0 deleted the fix/remove-snupkg branch January 28, 2026 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants