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

locale(en): remove word examples that contain spaces #281

Merged
merged 4 commits into from Aug 5, 2022

Conversation

griest024
Copy link
Contributor

@griest024 griest024 commented Jan 24, 2022

"word" to me implies that the string does not contain spaces. This PR removes all word examples that contain spaces other than boo hoo which was changed to boohoo

related to #276

@griest024 griest024 requested a review from a team as a code owner January 24, 2022 19:22
pkuczynski
pkuczynski previously approved these changes Jan 24, 2022
Copy link
Member

@pkuczynski pkuczynski left a comment

Choose a reason for hiding this comment

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

We could extend conjunction list with https://7esl.com/conjunctions-list/ (skipping those which contain spaces)

@griest024
Copy link
Contributor Author

We could extend conjunction list with https://7esl.com/conjunctions-list/ (skipping those which contain spaces)

done

import-brain
import-brain previously approved these changes Jan 25, 2022
@Shinigami92 Shinigami92 added the c: refactor PR that affects the runtime behavior, but doesn't add new features or fixes bugs label Mar 15, 2022
@Shinigami92
Copy link
Member

@griest024 Tests are running red, I assume you need to update the expected seeded values due to index changes

Copy link
Member

@xDivisionByZerox xDivisionByZerox left a comment

Choose a reason for hiding this comment

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

tests need updates

@xDivisionByZerox xDivisionByZerox added the needs test More tests are needed label Jun 17, 2022
@xDivisionByZerox xDivisionByZerox added m: word Something is referring to the word module c: locale Permutes locale definitions labels Jul 28, 2022
@ST-DDT ST-DDT removed the c: refactor PR that affects the runtime behavior, but doesn't add new features or fixes bugs label Aug 5, 2022
@ST-DDT ST-DDT removed the needs test More tests are needed label Aug 5, 2022
@ST-DDT
Copy link
Member

ST-DDT commented Aug 5, 2022

I updated this so we can either merge or discard it.

@ST-DDT ST-DDT added the s: needs decision Needs team/maintainer decision label Aug 5, 2022
@ST-DDT ST-DDT requested review from a team and Shinigami92 August 5, 2022 13:39
@codecov
Copy link

codecov bot commented Aug 5, 2022

Codecov Report

Merging #281 (be6ddce) into main (01cacd5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head be6ddce differs from pull request most recent head 1295c43. Consider uploading reports for the commit 1295c43 to get more accurate results

@@           Coverage Diff           @@
##             main     #281   +/-   ##
=======================================
  Coverage   99.62%   99.62%           
=======================================
  Files        2155     2155           
  Lines      236901   236885   -16     
  Branches      991      991           
=======================================
- Hits       236016   236004   -12     
+ Misses        864      860    -4     
  Partials       21       21           
Impacted Files Coverage Δ
src/locales/en/word/noun.ts 100.00% <ø> (ø)
src/locales/en/word/conjunction.ts 100.00% <100.00%> (ø)
src/locales/en/word/interjection.ts 100.00% <100.00%> (ø)
src/modules/address/index.ts 99.81% <0.00%> (+<0.01%) ⬆️
src/modules/internet/user-agent.ts 83.59% <0.00%> (+1.05%) ⬆️

@ST-DDT ST-DDT changed the title fix: remove word examples that contain spaces locale(en): remove word examples that contain spaces Aug 5, 2022
@ST-DDT ST-DDT enabled auto-merge (squash) August 5, 2022 17:46
@ST-DDT ST-DDT merged commit 0668cfe into faker-js:main Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: locale Permutes locale definitions m: word Something is referring to the word module p: 1-normal Nothing urgent s: needs decision Needs team/maintainer decision
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

6 participants