Skip to content
This repository has been archived by the owner on Dec 27, 2023. It is now read-only.

[Level 4] Update .traivs.yml #236

Merged
merged 2 commits into from
Apr 14, 2015
Merged

Conversation

jatindhankhar
Copy link
Contributor

Added SAUCE specific variables

Added SAUCE specific variables
@jatindhankhar jatindhankhar changed the title Update .traivs.yml [Level 4] Update .traivs.yml Apr 14, 2015
- secure: BOfaPuGkeDJX+YhpoZn7zXzZLs+Kb6GALuap7jRX33M6igamx+MCMl1OTtKRuFiosZBDZQrs/E24QzRzSF9gBzqSqPTgQNUkoECwaIm0IIsdA8ff1SoFuFw8x8iFKFPXVb5C72xtwqXfqRAqchX6Jd4v/Wh+mEDMESxE5S5XIRY=
#SAUCE_URL=http://jatindhankhar:********-****-****-****-************@ondemand.saucelabs.com/wd/hub
- secure: sWkxx3v8RxChO0lZSYqNjkYvfg1Re6LwpDeBZIgw1qBBBg5UdCLJZitOAAqKsdoUpeVD0Hk9o6UdWfVnYAw81xArvHpl32VQgItY60UVhHVoszX5CQFo7bp0p59dPZbzx4XjJfZhM0btrHOUHpJkeoU6Ia9Z+gTiZ04uC+Jqi/s=
#CODECLIMATE_REPO_TOKEN=**************************(smcgregor/privly-applications)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You duplicated a comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops 😕, Sorry
Removing it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No worries. After you remove the duplicate, please let me know when I should merge this. I am not permissioned for your Travis secure environmental variables.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just did that
Waiting for the build to yield green on both ends
I have a little question,
Travis builds will eat up my monthly free minutes?
Concerned because if quota is out, tests may fail

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but I am concerned about SauceLabs, they have free 200 minutes

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saw that while signing up for Sauce Labs, I thought I should have a big Open source project before applying .
So I should change plans explicitly under billing to free Open Sauce $0 plan or would they change the plan by themselves once premium minutes are over ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed the duplicated comment
@jatindhankhar
Copy link
Contributor Author

Build passing on my end Build Status
image

Can be merged 😄

smcgregor added a commit that referenced this pull request Apr 14, 2015
Added travis variables for jatindhankhar
@smcgregor smcgregor merged commit c8c5646 into privly:master Apr 14, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants