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.

Sign up for GitHub

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

Jump to bottom

Fix for syntax error discarding #35

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

Fix for syntax error discarding #35

wants to merge 4 commits into from

Commits on Apr 11, 2018

  1. Discarded offences with syntax er. are now handled 

    Offences with syntax error were dropped, now it is fixed by handling them. The return value of the method is the same as before but syntax error offences are included too if any exists.
    @europ
    europ committed Apr 11, 2018
    Configuration menu
    Copy the full SHA
    35256d2 View commit details
    Browse the repository at this point in the history

  2. Enabled method "allow" in spec 

    Due to "undefined method `allow' for #<RSpec::XX>" this line had to be removed.
    @europ
    europ committed Apr 11, 2018
    Configuration menu
    Copy the full SHA
    b727a42 View commit details
    Browse the repository at this point in the history

  3. Method "inspect" was divided into two parts. 

    The functionality of one method was divided into two logically functioning parts. One method obtain offences and the other one inspects these offences.
    @europ
    europ committed Apr 11, 2018
    Configuration menu
    Copy the full SHA
    970bc1f View commit details
    Browse the repository at this point in the history

  4. New tests added for testing syntax er. detection 

    New tests were added which are testing if any of syntactic error offences is dropped.
    @europ
    europ committed Apr 11, 2018
    Configuration menu
    Copy the full SHA
    c0e39a9 View commit details
    Browse the repository at this point in the history