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

Fixed an incorrect exception argument. #915

Merged
merged 1 commit into from
Feb 3, 2022
Merged

Fixed an incorrect exception argument. #915

merged 1 commit into from
Feb 3, 2022

Conversation

FoxTes
Copy link
Contributor

@FoxTes FoxTes commented Feb 3, 2022

Hello. I want to help the project.
I noticed the duplication of exceptions and corrected this section of the code.

@martincostello martincostello added this to the v7.3.0 milestone Feb 3, 2022
@martincostello martincostello changed the base branch from master to v724-or-v730 February 3, 2022 12:57
@martincostello martincostello merged commit 9d13eac into App-vNext:v724-or-v730 Feb 3, 2022
@martincostello
Copy link
Member

Thanks!

@FoxTes
Copy link
Contributor Author

FoxTes commented Feb 3, 2022

I've seen a lot of little things. I can look for problem areas.

@martincostello
Copy link
Member

Sure - open further PRs if you find any other issues worth fixing. For future PRs, please base them off the v724-or-v730 branch as noted in the Pull Request template, not the master branch please.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants