-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
IGNITE-15779 .NET: Fix release and nightly builds #9508
base: master
Are you sure you want to change the base?
Conversation
ivandasch
commented
Oct 21, 2021
•
edited by ptupitsyn
Loading
edited by ptupitsyn
- Fix build.ps1 to pack example sources without building them
- Fix nightly build version for nuget packages
- Add explicit PackageId to csproj files
f832322
to
19bb961
Compare
19bb961
to
89efe04
Compare
// This function cuts excessive part of version. | ||
function setCutClientVersion(dest, source) { | ||
var value = project.getProperty(source); | ||
var digitVer = /\d+\.\d+\.\d+/.exec(value); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure it is correct to throw away a part of the version like this. Especially for nightly builds, we should pack a pre-release version, not a release one. Pre-release version is any version with a suffix like -nightly
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not possible, unfortunately. AssemblyVersion must contain at most 4 positive numbers, less than max uint16.
This regex just throw away current date, but it is preserved in zip archive of nugets. AFAIK nugets are published to some private repo (myget), so is it the real problem?