Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

document Travis CI automated tests configuration without Sauce Labs #2619

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lemoustachiste
Copy link

Hi there,

I did struggle yesterday setting up automated testing on my PRs for my polymer project, so I thought adding the findings in the documentation would help more people finding the information quickly.

I am not going into too much details because I haven't experienced the need for it, but adding something about the custom wct.conf.json file could also be beneficial, for the reasons mentioned in this issue comment: https://github.com/Polymer/web-component-tester/issues/656#issuecomment-374068089

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

@lemoustachiste
Copy link
Author

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@ghost ghost self-requested a review June 1, 2018 02:47
@ghost ghost self-assigned this Jun 1, 2018
@ghost
Copy link

ghost commented Jun 1, 2018

Thanks! I will take a look as soon as possible

@ghost
Copy link

ghost commented Jun 13, 2018

@aomarks Can I have a quick sanity-check on this from you if you have a chance? Just for technical info, not style etc

@aomarks
Copy link
Member

aomarks commented Jun 13, 2018

Seems fine to me, but @azakus is more familiar with the details of how to get headless working.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants