-
-
Notifications
You must be signed in to change notification settings - Fork 609
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
Attach base statement as child for asTry dumpInfo #2584
Conversation
At least one commit author (n8.fitzgerald@rallyhealth.com) is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user |
I have signed the CLA |
Thanks! If want to change the email address used in the commit, you may need to do something like this:
where If you want to avoid exposing your email address read https://docs.github.com/en/account-and-profile/setting-up-and-managing-your-personal-account-on-github/managing-email-preferences/setting-your-commit-email-address. |
There are no incompatible changes |
There are no incompatible changes |
@Mergifyio refresh |
✅ Pull request refreshed |
There are no incompatible changes |
The
getDumpInfo
method ofasTry
should have the action that is to be tried as a child for logging purposes.This is my first contribution here, sorry if there's a template I need to be following or something. I created this issue for the problem and this single line is all that's needed for the solution. I am happy to add tests if reviewers think it appropriate but I followed exactly the convention established in
CleanUpAction