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

Updated name.ts #293

Merged
merged 2 commits into from
Oct 27, 2022
Merged

Updated name.ts #293

merged 2 commits into from
Oct 27, 2022

Conversation

simransia
Copy link
Contributor

I have added a few new pet names of snake, duck, and crab.

Kindly review this pull request and consider it as a hacktoberfest contribution. Please let me know in case any changes are required. Thank You :)

I have added a few new pet names of snake, duck, and crab.
'Medusa',
'Naga',
'Severus',
'Sir Hiss',
Copy link
Owner

Choose a reason for hiding this comment

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

Sir Hiss already appears on line 363, please remove

@@ -287,6 +287,33 @@ export const CRAB_NAMES: ReadonlyArray<string> = [
'Snapper',
'Coconut',
'Sebastian',
'Abby',
'Bubbles',
'Backhoe',
Copy link
Owner

Choose a reason for hiding this comment

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

Please remove Backhoe

@tonybaloney
Copy link
Owner

Thanks for the PR! Please make the required changes.

Also you need to run npm run lint:fix in the repository as some of the lines have trailing spaces after the commas (or fix them by hand).

@codecov-commenter
Copy link

Codecov Report

Merging #293 (5be399f) into master (1065478) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #293   +/-   ##
=======================================
  Coverage   10.53%   10.53%           
=======================================
  Files           5        5           
  Lines         655      655           
  Branches       92       92           
=======================================
  Hits           69       69           
  Misses        586      586           
Impacted Files Coverage Δ
src/common/names.ts 78.57% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tonybaloney tonybaloney merged commit 954634f into tonybaloney:master Oct 27, 2022
tonybaloney added a commit that referenced this pull request Feb 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants