8000 Enabled sourcelink and deterministic build by 304NotModified · Pull Request #5 · YuraSidorets/NLog.Targets.ActiveMQ · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Enabled sourcelink and deterministic build #5

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

Merged
merged 1 commit into from
Apr 24, 2025

Conversation

304NotModified
Copy link
Contributor
@304NotModified 304NotModified commented Apr 24, 2025
  • Enabled sourcelink and deterministic build
  • Push symbol packages (needed for sourcelink)
  • Bump version

Reason

image

TODO

  • check package

@304NotModified
Copy link
Contributor Author

@YuraSidorets please approve the build :)

@304NotModified 304NotModified force-pushed the sourcelink branch 2 times, most recently from a25745b to 37fe63c Compare April 24, 2025 21:03
@304NotModified
Copy link
Contributor Author

@YuraSidorets build should be fixed now

@304NotModified
Copy link
Contributor Author

This looks good

Successfully created package '/home/runner/work/NLog.Targets.ActiveMQ/NLog.Targets.ActiveMQ/NLog.Targets.ActiveMQ/NLog.Targets.ActiveMQ/bin/Release/NLog.Targets.ActiveMQ.2.1.2.nupkg'.
Successfully created package '/home/runner/work/NLog.Targets.ActiveMQ/NLog.Targets.ActiveMQ/NLog.Targets.ActiveMQ/NLog.Targets.ActiveMQ/bin/Release/NLog.Targets.ActiveMQ.2.1.2.snupkg'.

@304NotModified
Copy link
Contributor Author

I think is now ready for merge

@YuraSidorets YuraSidorets merged commit a8a98f7 into YuraSidorets:master Apr 24, 2025
1 check passed
@304NotModified 304NotModified deleted the sourcelink branch April 24, 2025 21:28
@304NotModified
Copy link
Contributor Author

@YuraSidorets

image

🎉

@YuraSidorets
Copy link
Owner

@YuraSidorets

image

🎉

@304NotModified Thank you! 🎉

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
0