Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add publish artifacts step to GitHub Actions workflow #82

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

lahma
Copy link
Contributor

@lahma lahma commented Aug 6, 2024

I noticed that the published NuGet package still had problems: https://nuget.info/packages/DeviceDetector.NET/6.3.3

Adding artifacts collection to GitHub action that allows to download properly built package for NuGet publish (both nupkg and snupkg should be published to NuGet.org).

Now if you download the artifacts from build summary: https://github.com/totpero/DeviceDetector.NET/actions/runs/10267431264?pr=82

NuGet package explorer gives all checks as green:

image

@lahma lahma force-pushed the build-artifacts branch from 442c40e to b4600ce Compare August 6, 2024 13:18
@lahma lahma force-pushed the build-artifacts branch from b4600ce to d5ea2bd Compare August 6, 2024 13:24
@lahma lahma marked this pull request as ready for review August 6, 2024 13:28
@totpero
Copy link
Owner

totpero commented Aug 8, 2024

Thank @lahma for your contributions 🥇

@totpero totpero merged commit 15d11e6 into totpero:master Aug 8, 2024
1 of 2 checks passed
@totpero totpero self-requested a review August 8, 2024 07:39
@totpero totpero added the detection improvements Pull requests that update a dependency file label Aug 8, 2024
@lahma lahma deleted the build-artifacts branch August 8, 2024 09:28
@lahma
Copy link
Contributor Author

lahma commented Nov 29, 2024

@totpero Thanks for the release of 6.4.1, seems that nuget.org doesn't have symbols though, could you please also upload the snupkg there?

Copy link
Owner

@totpero totpero left a comment

Choose a reason for hiding this comment

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

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
detection improvements Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants