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

🐛 Replace errors.As with Is #901

Merged
merged 1 commit into from
Aug 25, 2021
Merged

🐛 Replace errors.As with Is #901

merged 1 commit into from
Aug 25, 2021

Conversation

azeemshaikh38
Copy link
Contributor

  • Please check if the PR fulfills these requirements
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Replace errors.As calls with Is to avoid overwriting original error values. Thanks to @laurentsimon 🐛 Fixes for Branch Protection #900 (comment)

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    No.

Copy link
Contributor

@laurentsimon laurentsimon left a comment

Choose a reason for hiding this comment

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

That was fast!

@github-actions
Copy link

Integration tests success for ceb17bd239340a4b8574ff4e300f17f445feaffa

@github-actions
Copy link

Integration tests success for 9265201bcd2454ccc3fede368a26deb6adf8f9e1

@github-actions
Copy link

Integration tests success for 31a3284b9c054ea1628dcc5538c22362ceb81d4f

@github-actions
Copy link

Integration tests success for 43552f04309f50ebfe5b313b3920c026d891e75d

@github-actions
Copy link

Integration tests success for 34622e1e8915d8327281f467e1d59229fb4144b0

@azeemshaikh38 azeemshaikh38 enabled auto-merge (squash) August 25, 2021 00:43
@azeemshaikh38 azeemshaikh38 merged commit 41d0ce3 into main Aug 25, 2021
@azeemshaikh38 azeemshaikh38 deleted the azeems/errors branch August 25, 2021 01:03
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.

3 participants