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

task(test): setup travis #5

Merged
merged 3 commits into from
Jul 13, 2016
Merged

task(test): setup travis #5

merged 3 commits into from
Jul 13, 2016

Conversation

gnomeontherun
Copy link
Contributor

Description

Added a first draft of testing using Travis. Lots of room to improve over time.

What's included?

  • Travis CI configuration for unit tests with karma
  • Runs only in latest Firefox
Screenshots or link to CodePen/Plunker/JSfiddle

https://travis-ci.org/Teradata/covalent/builds/144336920

@kyleledbetter kyleledbetter added this to the Alpha 0.5 milestone Jul 13, 2016
@kyleledbetter
Copy link
Contributor

Awesome! Can you include the readme badge in this?

@gnomeontherun
Copy link
Contributor Author

Updated.

@emoralesb05
Copy link
Contributor

LGTM!

@@ -1,5 +1,7 @@
# Teradata Web Platform

[![Build Status](https://travis-ci.org/Teradata/covalent.svg?branch=feature%2Ftravis-ci)](https://travis-ci.org/Teradata/covalent)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be pointing to branch master or develop?

@kyleledbetter kyleledbetter merged commit 884823f into develop Jul 13, 2016
@emoralesb05 emoralesb05 deleted the feature/travis-ci branch July 13, 2016 04:24
@kyleledbetter kyleledbetter mentioned this pull request Jul 14, 2016
emoralesb05 pushed a commit that referenced this pull request Jul 23, 2019
* bump version

* fix(aot): Fix AOT Build

Can't use styles attribute on AOT, need to inject the css

* Removing use of Pipe for injecting css

* bump version

* Removing ViewEncapsulation.None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants