Skip to content

Remove Travis cI#260

Merged
SeanHealy33 merged 4 commits into
masterfrom
seanhealy/removeTravis
Aug 10, 2018
Merged

Remove Travis cI#260
SeanHealy33 merged 4 commits into
masterfrom
seanhealy/removeTravis

Conversation

@SeanHealy33

Copy link
Copy Markdown
Contributor

Why?

Removing Travis so we only use circle ci

jonnyom
jonnyom previously approved these changes Aug 10, 2018

@GabrielAnca GabrielAnca 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.

There's a travis badge in the README file that should be removed too

@SeanHealy33

Copy link
Copy Markdown
Contributor Author

Good catch @GabrielAnca

jonnyom
jonnyom previously approved these changes Aug 10, 2018
GabrielAnca
GabrielAnca previously approved these changes Aug 10, 2018

@GabrielAnca GabrielAnca 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.

🎉

Comment thread README.md
Climate](https://codeclimate.com/repos/537da4a7e30ba062b101be9c/badges/2aa25d4736f09f40282e/gpa.svg)](https://codeclimate.com/repos/537da4a7e30ba062b101be9c/feed) [![Build
Status](https://travis-ci.org/intercom/intercom-php.svg?branch=master)](https://travis-ci.org/intercom/intercom-php)

## intercom-php

@GabrielAnca GabrielAnca Aug 10, 2018

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.

Just double checking that this title was removed on purpose. It was redundant so I'm happy either with or without it 😄

@SeanHealy33
SeanHealy33 dismissed stale reviews from GabrielAnca and jonnyom via 025a9e6 August 10, 2018 11:21
@SeanHealy33
SeanHealy33 merged commit f782957 into master Aug 10, 2018
@SeanHealy33
SeanHealy33 deleted the seanhealy/removeTravis branch August 10, 2018 11:24
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