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

emoji layer company-emoji font support #3864

Closed
wants to merge 2 commits into from

Conversation

xcodebuild
Copy link
Contributor

Font support for company-emoji on Linux or on Mac OS X and using the Cocoa version of Emacs.
See https://github.com/dunn/company-emoji/blob/master/README.md

Font support for company-emoji on Linux or on Mac OS X and using the
Cocoa version of Emacs.
See https://github.com/dunn/company-emoji/blob/master/README.md
(progn
(setq company-emoji-insert-unicode nil)
;; From https://github.com/dunn/company-emoji/README.md for Linux, or on Mac OS X and using the Cocoa version of Emacs
(defun --set-emoji-font (frame)
Copy link
Contributor

Choose a reason for hiding this comment

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

If it's only useful in gui, wouldn't adding a guard when display-graphic-p in the function avoid it setting unnecessarily?

Copy link
Contributor

Choose a reason for hiding this comment

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

Wasn't there a more useful function than display-graphic-p? so that this also works when spawned in daemon mode? @travisbhartwell

@robbyoconnor
Copy link
Contributor

@codefalling why did you close this?

@robbyoconnor
Copy link
Contributor

it is very common for pull requests to take awhile to be merged...

@xcodebuild
Copy link
Contributor Author

@robbyoconnor This pr got a strange failing at travis-ci,how to fix it?

@robbyoconnor
Copy link
Contributor

@codefalling it might have been a git checkout taking too long to return...we don't have tests for layers currently and your change wouldn't impact Travis CI

@robbyoconnor
Copy link
Contributor

Go to the build details and look at the output -- most of the time it's not anyone's fault -- Travis CI sometimes just gives up

@xcodebuild xcodebuild reopened this Jan 30, 2016
@xcodebuild
Copy link
Contributor Author

There is a happy accident that closing and reopening pr would force Travis CI to rebuild 😄

@robbyoconnor
Copy link
Contributor

Yes, closing/re-opening a pull request triggers a Travis CI build.

@jb55
Copy link
Contributor

jb55 commented Dec 8, 2016

[codetriage] status of this?

@syl20bnr syl20bnr added the Merged label May 8, 2017
@syl20bnr syl20bnr self-assigned this May 8, 2017
@syl20bnr
Copy link
Owner

syl20bnr commented May 8, 2017

Thank you ! 👍
I made some modification in the PR, see 37f4a525
Cherry-picked into develop branch, you can safely delete your branch.

@syl20bnr syl20bnr closed this May 8, 2017
@syl20bnr syl20bnr removed the Merged label May 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants