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

Fixes for byte compiler & update travis #338

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

jabranham
Copy link
Contributor

Various fixes so the byte compiler is satisfied, a few checkdoc fixes.

string-as-unibyte is obsolete in upcoming Emacs 26, Can you double-check that (encode-coding-string data 'no-conversion) is right?

@jabranham
Copy link
Contributor Author

Rebased onto master @vermiculus

@vermiculus vermiculus self-assigned this Feb 9, 2018
@vermiculus
Copy link
Owner

Sorry I've been absent; I'll take a look at this by tomorrow night.

@jabranham
Copy link
Contributor Author

No problem, and no rush!

@jabranham
Copy link
Contributor Author

Rebased, and fixed a few more things the byte compiler was complaining about

@jabranham
Copy link
Contributor Author

Pushed a commit to make travis error on compiler warnings

@jabranham jabranham changed the title Fixes for byte compiler Fixes for byte compiler & update travis Feb 16, 2018
@jabranham jabranham closed this Jul 24, 2018
@vermiculus
Copy link
Owner

Turns out I lied ☹️

Do you mind if I keep this open anyway?

@jabranham
Copy link
Contributor Author

Yes, sorry I didn't mean to close. Was looking at old lingering PRs and must've hit the wrong button.

@jabranham jabranham reopened this Jul 24, 2018
@vermiculus
Copy link
Owner

Is this something that's still valuable to merge after all this time? Happy to do so if so.

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.

2 participants