-
Notifications
You must be signed in to change notification settings - Fork 15
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
Fix commit message #39
base: main
Are you sure you want to change the base?
Conversation
@KyleTryon For this one I've pulled fresh from your |
Any updates on this one? I'm eager to start using this when possible. Let me know if there is anything I can do to help. Thanks! |
Ive been using fpcs/[email protected] since this PR isnt merged yet. There is only issue i found so far. The notify step isnt executed on fail -> which is bad. It seams to be fixed on this branch though since the cotify command is set to run always. @mattrabe - really looking forward to using this update. |
@FellowFellow I have published v2.1.1 of my fork, which includes all of v2.1.0, plus the changes from this PR |
Clever! Thanks for the addition! Testing it out now, but it looks good. |
It may be a consequence of how we are checking out the code that maybe the parent is not being found.
|
@KyleTryon any updates on this? Due to some restrictions I can't use the forked orb of @mattrabe, so it would be awesome to have this merged 😬 let me know if I can help |
Sorry. I am no longer with the company. I would recommend reaching out to support. |
Fixes #37