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 ci step to do a plain dotnet build #10306

Merged
merged 4 commits into from
Oct 21, 2020
Merged

Conversation

brettfo
Copy link
Member

@brettfo brettfo commented Oct 21, 2020

While investigating #10151 I decided it would be a good idea to ensure that dotnet build ./FSharp.sln always works following an initial ./build.sh.

@brettfo brettfo force-pushed the plain-build branch 2 times, most recently from 81ccd84 to 815c890 Compare October 21, 2020 18:42
@auduchinok
Copy link
Member

@brettfo Could it also have a Windows build, in addition to Linux and macOS ones, please?

@KevinRansom
Copy link
Member

@auduchinok , it doesn't work on Windows ... I have to make some changes ....: #10151 (comment)

@auduchinok
Copy link
Member

@KevinRansom My suggestion is about having it in CI at least when we have a fix, so it doesn't accidentally get broken again. 🙂

@KevinRansom
Copy link
Member

@auduchinok , when I submit the fix, I will include a ci leg for it :-)

@brettfo
Copy link
Member Author

brettfo commented Oct 21, 2020

Since I'm already here I added a Windows version that can simply be un-commented when ready.

@KevinRansom
Copy link
Member

@brettfo thanks mate.

Copy link
Contributor

@cartermp cartermp left a comment

Choose a reason for hiding this comment

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

dope

@brettfo brettfo merged commit ba4d774 into dotnet:main Oct 21, 2020
@brettfo brettfo deleted the plain-build branch October 21, 2020 21:07
nosami pushed a commit to xamarin/visualfsharp that referenced this pull request Feb 23, 2021
* add ci step to do a plain `dotnet build`

* report installed SDK versions

* generate binlogs of each build

* add windows build
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