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

Use Data.Text instead of ByteString in simple-grep example to match the Text usage in Iris.Colour.Formatting #94

Closed
lillycham opened this issue Oct 20, 2022 · 1 comment · Fixed by #95
Assignees
Labels
enhancement New feature or request

Comments

@lillycham
Copy link
Contributor

No description provided.

@lillycham
Copy link
Contributor Author

Since this is dependent on my other PR #91, I'll open a PR for this when #91 is merged

@chshersh chshersh added the enhancement New feature or request label Oct 21, 2022
@chshersh chshersh added this to the v0.1.0.0: Improved UX milestone Jan 15, 2023
chshersh pushed a commit that referenced this issue Jan 15, 2023
… API (#95)

* Change: ByteString > Text in Iris.Colour.Formatting
* Update changelog to reflect changes in Iris.Colour.Formatting API
* Remove: bytestring dependency
* Move #90 unto unreleased section of changelog
* Remove unneeded import in Iris.Colour.Formatting
* Modify grep example to use Data.Text
* remove redundant imports
* replace bs.readfile with text
* correct mdlint errors
* Update changelog, remove bytestring dependency from grep.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants