Skip to content

Conversation

Br3nda
Copy link
Contributor

@Br3nda Br3nda commented Mar 13, 2016

No description provided.

@MartinHjelmare
Copy link
Collaborator

Since your other PR is merged now, you should rebase this one off dev branch. Please also sqaush your commits into one. Sorry for the runaround.

For continuous integration on Travis-CI
@Br3nda
Copy link
Contributor Author

Br3nda commented Mar 15, 2016

Squashed to one commit

@MartinHjelmare
Copy link
Collaborator

👍
I'd like to merge this. But even better would be to also activate travis for this repo. What do you say, @theolind ?

@theolind
Copy link
Owner

I think this is great and have enabled travis for the repository. However, I think "travis.yml" needs to be renamed to ".travis.yml" for travis to pick it up and use it.

@MartinHjelmare
Copy link
Collaborator

@theolind Great! Regarding the name, yeah it has to be named like that, and I think it is, it's just the commit message that reads a bit different.

I'm merging this now.

MartinHjelmare added a commit that referenced this pull request Mar 15, 2016
@MartinHjelmare MartinHjelmare merged commit 65c56c7 into theolind:dev Mar 15, 2016
@MartinHjelmare
Copy link
Collaborator

Build Status

@Br3nda Br3nda deleted the travis branch March 16, 2016 03:19
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