-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
[#281] Add example Cabal and Stack projects #422
Conversation
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.
Amazing work! Thank you so much 😍 Now we have ready go-to example to showcase!
README.md
Outdated
support, default settings and all of the integrations disabled. | ||
* `cabal-full`: Cabal-only project with all integrations enabled. | ||
* `stack-full`: Stack-only project with all integrations enabled. | ||
* `full-batteries`: All batteries-included project which supports both |
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.
Do you think it makes sense to make each of the folders as the link?
# Trigger the workflow on push or pull request, | ||
# but only for the master branch |
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.
Don't you mind putting this comment on one line?
runs-on: ubuntu-16.04 | ||
strategy: | ||
matrix: | ||
ghc: ["8.4.4", "8.6.5", "8.8.2"] |
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.
Maybe we should put each GHC version on a separate line here as well?
It will make it consistent with the other CI settings and also allows to change GHC versions easier.
91f705a
to
81bfe78
Compare
Resolves #281
I've created projects at GitHub to demonstrate that CI actually passes: