Skip to content

Add CI (appveyor) #15

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.

Sign up for GitHub

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

Jump to bottom
Merged
Parent: Update typo in README.md
merged 18 commits into from
Dec 12, 2019
Merged

Add CI (appveyor) #15

merged 18 commits into from
Dec 12, 2019

Conversation

@Mpdreamz
Copy link
Member

Lots of things in this PR

  • Sets up appveyor on commit (branches and PRs) canary builds
  • Canary builds runs all the tests
  • publishes when merging to integration branches to https://ci.appveyor.com/nuget/ecs-dotnet
  • All CI builds are always prereleases with (ci-DATE) suffix

Build somewhat simplified

  • packing of projects now goes through dotnet pack
  • Projects.fsx no longer needed
  • SourceLink using the source link package

@Mpdreamz
CI: tests and canary builds
5115869 
- Adds tests to build targets, picks up all tests projects under /tests
- Adds `scripts` to the solution.
- build.bat test simply tests all projects without other target
- Update fsproj file so it works on .net core sdk 3.0 and up
@Mpdreamz
clean up solution folders/items a bit, grouped more under RelatedFile…
299ceac 
…s/Build
@Mpdreamz
Move to dotnet pack
557af08
@Mpdreamz
package icon and url, scripts no longer pack and we validate artifact…
08e213c 
…s again
@Mpdreamz
remove the notion of Projects.fsx, handled in csproj files now
53662f0
@Mpdreamz
clean up Paths
5277fb6
@Mpdreamz
allow on nuget publishing on this branch temporarily
a31472e
@Mpdreamz
add azure-pipelines.yml
ec5ccac
@Mpdreamz
skip unit tests that does IO
359ce0f
@Mpdreamz
make sure tests build with version too
d9c3d3e
@Mpdreamz
remove no build from pack
2a1c149
@Mpdreamz
pass explicit version information to pack
7c9d883
@Mpdreamz
Azure devops linking is broken so removing that integration for now (…
8da8155 
…wanted to use coverage view)
@Mpdreamz
unlink azure-pipelines.yml from solution
47f3c39
@Mpdreamz
remove nuspec files
646e210
Mpdreamz
Mpdreamz commented Dec 11, 2019
View reviewed changes
@Mpdreamz Mpdreamz mentioned this pull request Dec 11, 2019
Closed
Mpdreamz
Mpdreamz commented Dec 12, 2019
View reviewed changes
@codebrain codebrain mentioned this pull request Dec 12, 2019
Closed
8 tasks
Mpdreamz
Mpdreamz commented Dec 12, 2019
View reviewed changes
Mpdreamz
Mpdreamz commented Dec 12, 2019
View reviewed changes
codebrain
codebrain approved these changes Dec 12, 2019
View reviewed changes
Copy link
Contributor

@codebrain codebrain left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@Mpdreamz Mpdreamz merged commit 5f3aaa0 into master Dec 12, 2019
@Mpdreamz Mpdreamz deleted the feature/ci branch December 12, 2019 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@codebrain codebrain codebrain approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Mpdreamz @codebrain