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.

Sign up for GitHub

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

Jump to bottom

Remove b script #27

Closed
vrom911 opened this issue Oct 11, 2017 · 2 comments
Closed

Remove b script #27

vrom911 opened this issue Oct 11, 2017 · 2 comments
Labels
bug script UX/UI design decisions
Milestone
v1.1: More config...

Comments

@vrom911
Copy link
Member

vrom911 commented Oct 11, 2017

As it seen at the screenshot coloring is not working as it should at the ghc 8.2.1:
image
@vrom911 vrom911 added UX/UI design decisions bug Hacktoberfest https://hacktoberfest.digitalocean.com/ script labels Oct 11, 2017
@chshersh chshersh removed the Hacktoberfest https://hacktoberfest.digitalocean.com/ label Dec 16, 2017
@chshersh chshersh mentioned this issue Feb 7, 2018
Merged
@chshersh
Copy link
Contributor

chshersh commented Mar 15, 2018 •
edited
Loading

The coloring and perl magic should be disabled for GHC >= 8.2.1. But also this should be added to stack.yaml:

ghc-options:
  "$locals": -fhide-source-paths

@chshersh
Copy link
Contributor

Okay, I propose to do the following with this b script:

  1. It makes sense only for older GHC and for multi-package projects. I think nobody cares about pretty output for GHC < 8.2.1 because such old GHCs are the default nowadays for creating new project.
  2. We should add "$locals" to every stack.yaml file.
  3. We should remove coloring from b script.

@chshersh chshersh mentioned this issue Aug 22, 2018
Closed
@chshersh chshersh changed the title Update b script for 8.2.1 ghc version Remove b script Aug 30, 2018
vrom911 added a commit that referenced this issue Aug 30, 2018
@vrom911
[#27] Remove b-script support
acf8137 
Improve README
Improve benchmarks cabal section
Remove 'module Main where' in benchmarks
@vrom911 vrom911 mentioned this issue Aug 30, 2018
Merged
chshersh pushed a commit that referenced this issue Aug 30, 2018
@vrom911 @chshersh
[#27] Remove b-script support (#143)
dd357bd 
Improve README
Improve benchmarks cabal section
Remove 'module Main where' in benchmarks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug script UX/UI design decisions
Projects
None yet
Milestone
v1.1: More configuration and new inte...
Development

No branches or pull requests
2 participants
@chshersh @vrom911