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

Fix load error message on Linux, fix documentation, and cleanup whitespace #3

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

Conversation

cvoltz
Copy link

@cvoltz cvoltz commented Mar 18, 2016

These patches:

  1. Fix load error on Linux

    On Linux systems, the DOS EOL terminators cause the plugin to fail to
    load. Replace the DOS EOL (LF CR) with UNIX EOL (LF) so the plugin will
    load correctly on both Linux and Windows. Also, remove trailing
    whitespace.

  2. Fix documentation

    Modify the text in the README and the plugin to be the same. Looks like
    they were originally the same but drifted apart over time.

    Fix several grammar and spelling mistakes.

    Modify layout of text in a few places to line things up better.

  3. Convert tabs to spaces

    Convert tabs to spaces so the files will look the same in all editors
    irrespective of the user's editor preferences. The original files used
    4 space tabs but that wasn't documented anywhere.

Christopher Voltz added 3 commits March 18, 2016 11:13
On Linux systems, the DOS EOL terminators cause the plugin to fail to
load. Replace the DOS EOL (LF CR) with UNIX EOL (LF) so the plugin will
load correctly on both Linux and Windows.  Also, remove trailing
whitespace.
Modify the text in the README and the plugin to be the same.  Looks like
they were originally the same but drifted apart over time.

Fix several grammar and spelling mistakes.

Modify layout of text in a few places to line things up better.
Convert tabs to spaces so the files will look the same in all editors
irrespective of the user's editor preferences.  The original files used
4 space tabs but that wasn't documented anywhere.
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.

1 participant