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

Adding appveyor to the project #1066

Merged
merged 9 commits into from
Aug 26, 2015
Merged

Adding appveyor to the project #1066

merged 9 commits into from
Aug 26, 2015

Conversation

joaomdmoura
Copy link
Member

appveyor is a well known service that enable us to test AMS on windows environments in order to make sure we are supporting developers that use windows.

Eric-Guo and others added 6 commits August 18, 2015 09:52
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
Test caller line parsing and digesting
@joaomdmoura
Copy link
Member Author

I'm still working on this, I don't think it will pass on this first attempt, this appveyor setting might be a little tricky.
If you have some expertise here make yourself comfortable to help by commenting or sending a PR to appveyor branch.

thanks @bf4 give many help and suggestion, original PR #1014
Make testing suite running and pass in Windows
@Eric-Guo
Copy link
Contributor

I try to running testing at appveyor several time using appveyor branch, except ruby 2.2, all works.

For ruby 2.2, it stop at gem install bundler and lead timeout, I already open tickets 2867 to appveyor.

@joaomdmoura
Copy link
Member Author

Yup, I think that even so we are good to go, I'm removing 2.2 for now in order to merge it, please keep this up to date with the ticket you opened 😄

joaomdmoura added a commit that referenced this pull request Aug 26, 2015
Adding appveyor to the project
@joaomdmoura joaomdmoura merged commit af280ab into master Aug 26, 2015
@Eric-Guo
Copy link
Contributor

👍

@bf4 bf4 deleted the appveyor branch July 8, 2016 21:21
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.

3 participants