Skip to content

Conversation

@304NotModified
Copy link
Contributor

Current result:

image

Desired result:

Deterministic Build, see https://github.com/clairernovotny/DeterministicBuilds

Preview NuGet packages: https://ci.appveyor.com/project/304NotModified/nunit/builds/38580291/artifacts

And green check :)

image

@304NotModified
Copy link
Contributor Author

PS: I assumed the latest dev branch, but not sure, can't find that info - checked https://github.com/nunit/nunit/blob/master/CONTRIBUTING.md

@rprouse
Copy link
Member

rprouse commented Apr 7, 2021

It is fine to merge this to the 3.13 dev branch, but our default dev branch is actually master. We are working on the 4.0 release, but merging bug fixes and changes like this back to this 3.13 branch for any point releases. We haven't fully setup this branch properly for PR builds yet so I will need to Admin merge it and then we can cherry pick to master.

@rprouse rprouse self-assigned this Apr 7, 2021
Copy link
Contributor

@jnm2 jnm2 left a comment

Choose a reason for hiding this comment

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

Thanks for the help!

@rprouse rprouse merged commit f5a9fff into nunit:v3.13-dev Apr 12, 2021
@rprouse rprouse added this to the 3.13.2 milestone Apr 12, 2021
@304NotModified 304NotModified deleted the deterministic-build branch April 12, 2021 21:44
@304NotModified
Copy link
Contributor Author

FYI|

looks great!

image

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.

4 participants