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

Integrate Ebert #38

Merged
merged 5 commits into from Apr 23, 2017
Merged

Integrate Ebert #38

merged 5 commits into from Apr 23, 2017

Conversation

crbelaus
Copy link
Owner

This PR integrates Ebert into the build pipeline along with Travis.
While Travis keeps running the unit tests of the application, Ebert will check the style and potential bugs and improvements of the code.

As recommended by the style guide, pipe expressions should always start
in a literal instead of a function call
@crbelaus crbelaus self-assigned this Apr 23, 2017
@sourcelevel-bot
Copy link

Ebert has finished reviewing this Pull Request and has found:

  • 5 fixed issues! 🎉

You can see more details about this review at https://ebertapp.io/github/belaustegui/trans/pulls/38.

@crbelaus crbelaus merged commit f5db4f4 into master Apr 23, 2017
@crbelaus crbelaus deleted the integrate-ebert branch April 23, 2017 08:19
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.

None yet

1 participant