statebox / cql

CQL: Categorical Query Language implementation in Haskell
GNU Affero General Public License v3.0
162 stars 14 forks source link

fix styling according to stack --pedantic #38 #49

Closed marcosh closed 5 years ago

marcosh commented 5 years ago

I adapted the code to avoid errors while running stack with the pedantic flag.

I also added -Wall -Werror to package.yaml so that a more correct styling is enforced by the compiler

wisnesky commented 5 years ago

I think the result of me having to merge AQL.hs by hand was that GitHub just used the copy on my laptop - these changes may have been lost in that file. Instance.hs may be worth checking too

On Oct 9, 2018, at 3:25 AM, Marco Perone notifications@github.com wrote:

I adapted the code to avoid errors while running stack with the pedantic flag.

I also added -Wall -Werror to package.yaml so that a more correct styling is enforced by the compiler

You can view, comment on, or merge this pull request online at:

https://github.com/statebox/aql/pull/49

Commit Summary

• fix styling according to stack --pedantic #38 File Changes

• M app/Main.hs (11) • M package.yaml (4) • M src/Language/AQL.hs (169) • M src/Language/Common.hs (19) • M src/Language/Instance.hs (390) • M src/Language/Mapping.hs (144) • M src/Language/Options.hs (123) • M src/Language/Parser.hs (70) • M src/Language/Parser/Generator/Generator.hs (3) • M src/Language/Parser/Instance.hs (33) • M src/Language/Parser/Mapping.hs (44) • M src/Language/Parser/Parser.hs (18) • M src/Language/Parser/Schema.hs (40) • M src/Language/Parser/Transform.hs (28) • M src/Language/Parser/Typeside.hs (29) • M src/Language/Program.hs (20) • M src/Language/Prover.hs (38) • M src/Language/Query.hs (8) • M src/Language/Schema.hs (107) • M src/Language/Term.hs (215) • M src/Language/Transform.hs (149) • M src/Language/Typeside.hs (19) Patch Links:

https://github.com/statebox/aql/pull/49.patchhttps://github.com/statebox/aql/pull/49.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

marcosh commented 5 years ago

@wisnesky I should have fixed everything with https://github.com/statebox/aql/pull/56