-
Notifications
You must be signed in to change notification settings - Fork 183
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
Use GitHub Actions for CI #1892
Use GitHub Actions for CI #1892
Conversation
8d5dbc3
to
a3f5291
Compare
a3f5291
to
1db1f00
Compare
2f0fadb
to
0ee38bf
Compare
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 content to merge once we remove AppVeyor requirement and rename.
Also, we probably want to merge the Tools changes first? I think the deployment script change from #1895 can wait until after this, no?
|
da3fa85
to
d937523
Compare
441bd94
to
5882014
Compare
5882014
to
8f8dddc
Compare
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.
Looks great, except for the docs (and how to build, where we should probably mention the build.ps1
.
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.
Thanks, @thomaslevesque. Great work!
build.cmd = build.cmd | ||
build.sh = build.sh |
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.
nice
This change has been released as part of FakeItEasy 7.4.0. |
No description provided.