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

Allow custom derived data path and ignored_files support #7

Merged
merged 5 commits into from
Mar 9, 2019

Conversation

KingOfBrian
Copy link

@KingOfBrian KingOfBrian commented Mar 6, 2019

I'm setting up danger-xcprofiler on a Jenkins box, and we put the derived data in a different directory. The plugin doesn't allow you to specify a DerivedData directory, but xcprofiler does. This just relays through another string.

Additionally, we have a handful of errors in our Pods directory that I want to be able to ignore. So I added an ignored_files property to the plugin that mimics the behavior of danger-xcode_summary and allows a simple regular expression.

@coveralls
Copy link

coveralls commented Mar 7, 2019

Coverage Status

Coverage increased (+0.1%) to 98.571% when pulling 2e50deb on KingOfBrian:custom_derived_data_path into 707163d on giginet:master.

@KingOfBrian KingOfBrian changed the title Allow the plugin to pass the derived data path through to xcprofiler. Allow custom derived data path and ignored_files support Mar 8, 2019
@giginet
Copy link
Owner

giginet commented Mar 8, 2019

Thanks! I'll check this tomorrow

Copy link
Owner

@giginet giginet left a comment

Choose a reason for hiding this comment

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

LGTM

Thank you for your contribution.
I'll release newer soon.

@giginet giginet merged commit 535b8fb into giginet:master Mar 9, 2019
@giginet
Copy link
Owner

giginet commented Mar 9, 2019

@KingOfBrian I just released v0.4.0. Thanks 😄

https://rubygems.org/gems/danger-xcprofiler

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.

None yet

3 participants