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

Build Statistic Reporting for TeamCity #150

Merged
merged 6 commits into from
Feb 23, 2016

Conversation

michaelmyers
Copy link
Contributor

This adds the flag --teamcity and then within the simple_output coverage service it leverages the default build statistics for coverage to report the results to TeamCity, a continuous integration server by JetBrains.

Example usage would be:

$ slather coverage --teamcity -s

We also can consider cherry-picking joshrlesch's commit from PR 132 which adds support for TeamCity within the coverall coverage service to complete the solution.

This is my first time with ruby so I am looking forward to feedback.

@kdawgwilk
Copy link

👍

@choli
Copy link

choli commented Feb 18, 2016

this works, thanks 👍

@@ -6,7 +6,7 @@
proj = Slather::Project.open(FIXTURES_PROJECT_PATH)
proj.build_directory = TEMP_DERIVED_DATA_PATH
proj.input_format = "profdata"
proj.configure
proj.send(:configure)
Copy link
Member

Choose a reason for hiding this comment

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

This seems unrelated to the PR. Leftover or am I missing something?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You are correct, this isn't needed.

@neonichu
Copy link
Member

@michaelmyers Thanks a lot for the PR 👍 Would you mind adding a Changelog entry?

@michaelmyers
Copy link
Contributor Author

@neonichu removed unnecessary code and added a changelog entry. Let me know if you need anything else. Thanks for your work on the project!

@neonichu
Copy link
Member

Thanks, it's good to go, will merge once Travis is 💚

neonichu added a commit that referenced this pull request Feb 23, 2016
Build Statistic Reporting for TeamCity
@neonichu neonichu merged commit f4d2e3b into SlatherOrg:master Feb 23, 2016
@shritekale
Copy link

Does anyone know when this would be released?

@michaelmyers
Copy link
Contributor Author

This does not answer your question but if you want to use it now you can use:

gem 'slather', :git => 'git://github.com/SlatherOrg/slather', ref: 'f4d2e3b'

in your Gemfile.

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.

5 participants