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

fix methodArn #1283

Closed
wants to merge 1 commit into from
Closed

fix methodArn #1283

wants to merge 1 commit into from

Conversation

zoellner
Copy link
Contributor

@zoellner zoellner commented Oct 7, 2021

Description

undo 5ff81fb

Motivation and Context

solve #1043

How Has This Been Tested?

Screenshots (if appropriate):

@dherault
Copy link
Owner

Old and tests are failing, closing.

@dherault dherault closed this Apr 13, 2022
@zoellner
Copy link
Contributor Author

I disagree with this. The previous PR broke things. This one fixes them again.
#1043 is a real issue. #980 was a bad PR

@dherault
Copy link
Owner

Feel free to reopen it, I will merge it and publish it.

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

2 participants