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

Jump to bottom

Support proper parse the file name in mingw platform. #1014

Closed
wants to merge 4 commits into from
Closed

Support proper parse the file name in mingw platform. #1014

wants to merge 4 commits into from

Commits on Aug 18, 2015

  1. need lookahead match for windows file path contain 'c:/git/' 

    So we will got full file path instead of only c if caller.first is: c:/git/emberjs/ember-crm-backend/app/serializers/lead_serializer.rb:1:in `<top (required)>'

CALLER_FILE = /
  /A # start of string
  \S+ # one or more non-spaces
  (?= # stop previous match when
    :\d+:in # a colon is followed by one or more digits
               # followed by a colon followed by in
   )
/x

credit from https://gist.github.com/mikezter/540132 and @bf4
    @Eric-Guo
    Eric-Guo committed Aug 18, 2015
    Copy the full SHA
    f93a7e8 View commit details
    Browse the repository at this point in the history

  2. Test caller line parsing and digesting

    @bf4
    bf4 committed Aug 18, 2015
    Copy the full SHA
    215fb85 View commit details
    Browse the repository at this point in the history

Commits on Aug 19, 2015

  1. Merge pull request #1 from bf4/parse_windows_paths 

    Test caller line parsing and digesting
    @Eric-Guo
    Eric-Guo committed Aug 19, 2015
    Copy the full SHA
    90bff2f View commit details
    Browse the repository at this point in the history

Commits on Aug 21, 2015

  1. Make testing suite running and pass in Windows 

    thanks @bf4 give many help and suggestion
    @Eric-Guo
    Eric-Guo committed Aug 21, 2015
    Copy the full SHA
    4459652 View commit details
    Browse the repository at this point in the history