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

True color support #8

Merged
merged 18 commits into from
Dec 29, 2019
Merged

Conversation

guicho271828
Copy link
Contributor

@guicho271828 guicho271828 commented Dec 13, 2019

true-color (24bit) support, update to cl-colors2, better documentation, better source code organization, testing on more lisps

@pnathan
Copy link
Owner

pnathan commented Dec 13, 2019

hello @guicho271828 ! Thank you for your contribution! It appears that the docker system used for the unit tests is dead. You might have noticed this library hasn't been updated lately. :)

I am in the middle of a big deadline at work; I can try to get the UT fixed tonight or this weekend.

edit: whoops, I just noticed that you'd redone the UT system - I added comments and snippet.

.travis.yml Outdated Show resolved Hide resolved
Copy link
Owner

@pnathan pnathan left a comment

Choose a reason for hiding this comment

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

summary feedback: please make unit tests work - some other nitty bits.

Thanks for doing this. It'll be appreciated!

.travis.yml Show resolved Hide resolved
.travis.yml Show resolved Hide resolved
cl-ansi-text.asd Outdated Show resolved Hide resolved
src/define-colors.lisp Show resolved Hide resolved
src/define-colors.lisp Outdated Show resolved Hide resolved
.travis.yml Outdated Show resolved Hide resolved
@pnathan
Copy link
Owner

pnathan commented Dec 14, 2019 via email

@guicho271828
Copy link
Contributor Author

Done, Thank you for the prompt response. Also, added more comments about the SLIME REPL integration.

@guicho271828
Copy link
Contributor Author

ping

hope you are not working in the christmas
(but I do lol)

@pnathan pnathan merged commit d9b1ea3 into pnathan:master Dec 29, 2019
@pnathan
Copy link
Owner

pnathan commented Dec 29, 2019

Awesome! Sorry about the delay. I get distracted. >.>

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