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

feat: add music.songName #996

Merged
merged 3 commits into from
May 25, 2022
Merged

Conversation

wingleungchoi
Copy link
Contributor

Fixes: #162
Motivation: #162 (comment)

add faker.music.songName
contains the following special characters

  • ( and ' e.g. "(I Can't Get No) Satisfaction"

Top 1000 song list is based on https://manghammath.com/80s%20Hits/The%20Top%201000%20Songs%20of%20AllTime.pdf

@wingleungchoi wingleungchoi requested a review from a team as a code owner May 24, 2022 15:42
@codecov
Copy link

codecov bot commented May 24, 2022

Codecov Report

Merging #996 (b327d09) into main (c95826f) will decrease coverage by 0.00%.
The diff coverage is 99.51%.

@@            Coverage Diff             @@
##             main     #996      +/-   ##
==========================================
- Coverage   99.67%   99.67%   -0.01%     
==========================================
  Files        2112     2113       +1     
  Lines      225980   227001    +1021     
  Branches      980      982       +2     
==========================================
+ Hits       225237   226253    +1016     
- Misses        723      728       +5     
  Partials       20       20              
Impacted Files Coverage Δ
src/definitions/music.ts 0.00% <0.00%> (ø)
src/locales/en/music/index.ts 100.00% <100.00%> (ø)
src/locales/en/music/song_name.ts 100.00% <100.00%> (ø)
src/modules/music/index.ts 100.00% <100.00%> (ø)

@import-brain import-brain added c: feature Request for new feature p: 1-normal Nothing urgent labels May 24, 2022
@import-brain import-brain added this to the v7 - Current Major milestone May 24, 2022
import-brain
import-brain previously approved these changes May 24, 2022
@import-brain import-brain requested a review from a team May 24, 2022 17:04
src/definitions/music.ts Outdated Show resolved Hide resolved
Co-authored-by: ST-DDT <ST-DDT@gmx.de>
@ST-DDT ST-DDT requested review from a team May 24, 2022 18:58
@Shinigami92 Shinigami92 changed the title feat: songName in faker.music (close #162) feat: add music.songName May 25, 2022
@Shinigami92 Shinigami92 enabled auto-merge (squash) May 25, 2022 17:01
@Shinigami92 Shinigami92 merged commit ccd2959 into faker-js:main May 25, 2022
Minozzzi pushed a commit to Minozzzi/faker that referenced this pull request Jul 19, 2022
Co-authored-by: Wing Leung Choi <wchoi@and.digital>
@xDivisionByZerox xDivisionByZerox added m: music Something is referring to the music module c: locale Permutes locale definitions labels Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: feature Request for new feature c: locale Permutes locale definitions m: music Something is referring to the music module p: 1-normal Nothing urgent
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Adding new feature songName in faker.music
5 participants