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

deleted: drop GHC 7 test #225

Merged
merged 2 commits into from
Sep 12, 2018
Merged

deleted: drop GHC 7 test #225

merged 2 commits into from
Sep 12, 2018

Conversation

ncaq
Copy link
Contributor

@ncaq ncaq commented Aug 25, 2018

Yesod drop support GHC 7.
yesod/.travis.yml at cdba6c1678d8002eac94ba175e00183a7e87c09d ·
yesodweb/yesod

GHC 7 is slow, and GHC 7 support add CPP switch.
To drop GHC 7 get correct travis test and simple code.

I delete unneed CPP pragma.

And I run stylish-haskell all source code.
Because I want to detect unneed import.

Copy link
Member

@snoyberg snoyberg left a comment

Choose a reason for hiding this comment

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

I've added two comments. Additionally: the CI is not currently passing.

Text/Hamlet.hs Outdated
@@ -154,7 +147,7 @@ mkConName :: DataConstr -> Name
mkConName = mkName . conToStr

conToStr :: DataConstr -> String
conToStr (DCUnqualified (Ident x)) = x
conToStr (DCUnqualified (Ident x)) = x
Copy link
Member

Choose a reason for hiding this comment

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

I'm not in favor of these alignment changes, they create large diffs in PRs unnecessarily.

@@ -82,9 +82,6 @@ library

extensions: TemplateHaskell

if impl(ghc >= 7.4)
Copy link
Member

Choose a reason for hiding this comment

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

There's no change in the cabal file to indicate that GHC 7 is no longer supported, e.g. base >= 4.9.

@ncaq ncaq force-pushed the delete-ghc-7 branch 6 times, most recently from 6ec7fc0 to 5cf32e0 Compare August 30, 2018 02:37
@ncaq
Copy link
Contributor Author

ncaq commented Aug 30, 2018

@snoyberg
I reduce diff.
Please code review.

I do alignment import list and pragma.
However I think that it is need diff.
Because I remove import branch by package version and I remove CPP pragma.

@snoyberg
Copy link
Member

I do alignment import list and pragma.
However I think that it is need diff.
Because I remove import branch by package version and I remove CPP pragma.

That's exactly why the arbitrary restyling and reordering should not be included here, I can't reliably tell what has and has not changed.

Yesod drop support GHC 7.
[yesod/.travis.yml at cdba6c1678d8002eac94ba175e00183a7e87c09d ·
yesodweb/yesod](https://github.com/yesodweb/yesod/blob/cdba6c1678d8002eac94ba175e00183a7e87c09d/.travis.yml)

GHC 7 is slow, and GHC 7 support add CPP switch.
To drop GHC 7 get correct travis test and simple code.

I delete unneed CPP pragma.

I rename `pattern` to `newPattern` because it cause hlint error.
@ncaq
Copy link
Contributor Author

ncaq commented Sep 8, 2018

@snoyberg
This time I should have fulfilled the requirement.

@snoyberg snoyberg merged commit 2644b1c into yesodweb:master Sep 12, 2018
@ncaq ncaq deleted the delete-ghc-7 branch September 13, 2018 07:18
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.

None yet

2 participants