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

Linguistic fixes of 'does not exists' #14542

Merged
merged 1 commit into from Dec 23, 2020
Merged

Linguistic fixes of 'does not exists' #14542

merged 1 commit into from Dec 23, 2020

Conversation

CSharperMantle
Copy link
Contributor

Fixes #14020.
Replaces "does not exists" with "does not exist" across the framework, in the hope of correcting PR #14024

@CSharperMantle CSharperMantle changed the title linguistic fixes of 'does not exists' Linguistic fixes of 'does not exists' Dec 23, 2020
@gwillcox-r7 gwillcox-r7 self-assigned this Dec 23, 2020
@gwillcox-r7
Copy link
Contributor

According to #14020 there should be a total of 21 files that need fixing however I only see 19 files as being edited here. I'll add a commit which fixes up the remaining 3 files that were not included.

@gwillcox-r7
Copy link
Contributor

According to #14020 there should be a total of 21 files that need fixing however I only see 19 files as being edited here. I'll add a commit which fixes up the remaining 3 files that were not included.

Nm I'm an idiot some of these files are repeats of one another.

@gwillcox-r7
Copy link
Contributor

gwillcox-r7 commented Dec 23, 2020

Okay confirmed this fixes all instances of the does not exists. Sorry for the confusion @CSharperMantle and congrats on your first contribution to Metasploit! Landing this now 🥳

@gwillcox-r7 gwillcox-r7 merged commit 5565b23 into rapid7:master Dec 23, 2020
@CSharperMantle CSharperMantle deleted the csharpermantle-ling-patch branch December 23, 2020 22:46
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.

Fix all the "does not exists"
2 participants