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

[#281] Add example Cabal and Stack projects #422

Merged
merged 5 commits into from
Feb 24, 2020

Conversation

chshersh
Copy link
Contributor

Resolves #281

I've created projects at GitHub to demonstrate that CI actually passes:

@chshersh chshersh added tests Unit & Property tests docs Haddock, README, tutorials etc. labels Feb 23, 2020
@chshersh chshersh added this to the v2.0: Major update milestone Feb 23, 2020
@chshersh chshersh requested a review from vrom911 as a code owner February 23, 2020 17:25
@chshersh chshersh self-assigned this Feb 23, 2020
Copy link
Member

@vrom911 vrom911 left a 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
Copy link
Member

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?

Comment on lines 4 to 5
# Trigger the workflow on push or pull request,
# but only for the master branch
Copy link
Member

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"]
Copy link
Member

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.

@chshersh chshersh force-pushed the chshersh/281-Add-example-Cabal-and-Stack branch from 91f705a to 81bfe78 Compare February 24, 2020 18:35
@vrom911 vrom911 merged commit e666c66 into master Feb 24, 2020
@vrom911 vrom911 deleted the chshersh/281-Add-example-Cabal-and-Stack branch February 24, 2020 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Haddock, README, tutorials etc. tests Unit & Property tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add example Cabal and Stack projects
2 participants