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

Windows support #77

Merged
merged 2 commits into from
Mar 10, 2019
Merged

Windows support #77

merged 2 commits into from
Mar 10, 2019

Conversation

mystal
Copy link
Contributor

@mystal mystal commented Feb 10, 2019

Continuing the work from #59.

Bassetts and others added 2 commits February 5, 2019 09:32
We try to enable ANSI color support on Windows using the new APIs.
If that succeeds, we print colors as usual. If it fails, we disable
styles completely.
@dbrgn
Copy link
Collaborator

dbrgn commented Feb 18, 2019

@mystal thanks a lot for the PR! 🚀

I'm quite busy at the moment, but will try to take a look at the PR this week. Should I forget, please ping me!

@mystal
Copy link
Contributor Author

mystal commented Feb 25, 2019

@dbrgn Quick ping, as requested :)

@dbrgn dbrgn mentioned this pull request Mar 7, 2019
@dbrgn
Copy link
Collaborator

dbrgn commented Mar 7, 2019

Finally, looks good now! Thanks @mystal and sorry for the delay.

Last request, in #59 you wrote:

For now I suggest we tell devs on Windows to set their git config to check out tealdeer using Unix-style line endings.

I think this could be configured with the .gitattributes file, could you verify that?

@mystal
Copy link
Contributor Author

mystal commented Mar 9, 2019

Oh yeah, looks like a .gitattributes file was already added in the original PR. Should be good to go! 👍

@dbrgn
Copy link
Collaborator

dbrgn commented Mar 10, 2019

Great, let's merge this! 🙂 Thanks @mystal.

Next step would be getting #52 to run.

@dbrgn dbrgn merged commit 87ddb14 into tealdeer-rs:master Mar 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants