Skip to content

build: run chrome and firefox headless tests in circleci - #13603

Merged
josephperrott merged 1 commit into
angular:masterfrom
devversion:build/run-tests-on-circle
Oct 16, 2018
Merged

build: run chrome and firefox headless tests in circleci#13603
josephperrott merged 1 commit into
angular:masterfrom
devversion:build/run-tests-on-circle

Conversation

@devversion

@devversiondevversion commented Oct 13, 2018

Copy link
Copy Markdown
Member
  • Moves the travis_required TravisCI job to CircleCI.

Related ngcontainer PR: angular/angular#26435. This is not blocking us, since we use circleci/node-10.20-browsers to be able to use Firefox headless.

PR based and blocked on: #13596

@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 13, 2018
@devversion
devversionforce-pushed the build/run-tests-on-circle branch 4 times, most recently from 4cfc804 to d1be4aaCompareOctober 13, 2018 14:48
@devversiondevversion added blocked This issue is blocked by some external factor, such as a prerequisite PR pr: merge safe target: major This PR is targeted for the next major release labels Oct 13, 2018
Comment thread.circleci/config.yml
tests_local_browsers:
docker:
# TODO(devversion): Temporarily use a image that includes Firefox 62 because the
# ngcontainer image does include an old Firefox version that does not support headless.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did you / can you file an issue for this and reference it here?

@devversiondevversionOct 15, 2018

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

I actually filed a PR and chatted with Alex about it. Added the URL of the PR to the comment

@ngbot

ngbotBot commented Oct 15, 2018

Copy link
Copy Markdown

Hi @devversion! This PR has merge conflicts due to recent upstream merges.
Please help to unblock it by resolving these conflicts. Thanks!

* Moves the `travis_required` TravisCI job to CircleCI. Our `ngcontainer` image already comes with Chrome and Firefox installed.
@devversion
devversionforce-pushed the build/run-tests-on-circle branch from 8b160cc to 1b12618CompareOctober 15, 2018 16:57
@devversiondevversion removed the blocked This issue is blocked by some external factor, such as a prerequisite PR label Oct 15, 2018

@jelbournjelbourn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jelbournjelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Oct 16, 2018
@josephperrott
josephperrott merged commit 8e324c8 into angular:masterOct 16, 2018
roboshoes pushed a commit to roboshoes/material2 that referenced this pull request Oct 23, 2018
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 9, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreementtarget: majorThis PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@devversion@jelbourn@googlebot@josephperrott