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

Danger executor #224

Merged
merged 6 commits into from Apr 13, 2019

Conversation

Projects
None yet
2 participants
@f-meloni
Copy link
Member

commented Apr 13, 2019

We use ShellOut in some case and our executor in some others.
Given we have also some small performances problems, I thought would be a good idea to refactor the code to use just our internal shell executor

@orta

This comment has been minimized.

Copy link
Member

commented Apr 13, 2019

👌 less is less

👍

f-meloni added some commits Apr 13, 2019

@f-meloni

This comment has been minimized.

Copy link
Member Author

commented Apr 13, 2019

merge on green

@peril-staging peril-staging bot merged commit f2c0120 into master Apr 13, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.