Skip to content

[WIP] TravisCi Integration#536

Merged
Haehnchen merged 1 commit intoHaehnchen:masterfrom
Sorien:master
Jul 3, 2015
Merged

[WIP] TravisCi Integration#536
Haehnchen merged 1 commit intoHaehnchen:masterfrom
Sorien:master

Conversation

@Sorien
Copy link
Copy Markdown
Contributor

@Sorien Sorien commented Jun 29, 2015

No description provided.

@Sorien
Copy link
Copy Markdown
Contributor Author

Sorien commented Jun 29, 2015

@Haehnchen pls setup Travis for this repository

@Sorien Sorien changed the title TravisCi Integration [VIP] TravisCi Integration Jun 29, 2015
@Haehnchen Haehnchen closed this Jun 29, 2015
@Haehnchen Haehnchen reopened this Jun 29, 2015
@Haehnchen
Copy link
Copy Markdown
Owner

ok nice so far, missing plugins:

@Sorien
Copy link
Copy Markdown
Contributor Author

Sorien commented Jun 30, 2015

@Haehnchen there are some compilation errors can you try look at it?

@Sorien
Copy link
Copy Markdown
Contributor Author

Sorien commented Jul 1, 2015

1 Error, 4 Failed other tests running fine ... that's all I can do

There is one file with wrong name /assets/TwignamedAssetsServiceParser.java should be named /assets/TwigNamedAssetsServiceParser.java (i couldnt rename it directly on win commit dc843b2 and cf49266)

@Haehnchen
Copy link
Copy Markdown
Owner

thx, its working. 👍 removed the memory leak in a crazy commit :)
34c7553

Can you combine the commits into one? its then ready to merge

@Sorien
Copy link
Copy Markdown
Contributor Author

Sorien commented Jul 3, 2015

i had to rename TwigTranslationFixturesTest.java to TwigTranslationFixturesTestCase.java couse ant junit is not ignoring abstract classes

Haehnchen added a commit that referenced this pull request Jul 3, 2015
@Haehnchen Haehnchen merged commit 15223fb into Haehnchen:master Jul 3, 2015
@Haehnchen
Copy link
Copy Markdown
Owner

👍

@Sorien Sorien changed the title [VIP] TravisCi Integration [WIP] TravisCi Integration Jul 3, 2015
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.

2 participants