FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Remove duplicate 'Recalled to Life' book title by gkrimer · Pull Request #3238 · faker-ruby/faker · GitHub

Remove duplicate 'Recalled to Life' book title - #3238

Merged
stefannibrasil merged 1 commit into
faker-ruby:mainfrom
gkrimer:fix/remove-duplicate-recalled-to-life
Apr 14, 2026
Merged

Remove duplicate 'Recalled to Life' book title#3238
stefannibrasil merged 1 commit into
faker-ruby:mainfrom
gkrimer:fix/remove-duplicate-recalled-to-life

Conversation

gkrimer commented Apr 14, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Motivation / Background

Faker::Book.title can return "Recalled to Life" with twice the expected probability because it appears twice in the title list (lines 141–142 of lib/locales/en/book.yml).

The duplicate has been present since #484 (Dec 2015), when the book title list was bulk-replaced with titles sourced from Wikipedia. Both Reginald Hill's novel and Book 1 of Dickens' A Tale of Two Cities share the name, so the source list had two entries that were pasted without deduplication.

This PR removes one of the two identical entries.

Checklist

  • One change per PR
  • Detailed commit message
  • No CHANGELOG.md changes

The title has appeared twice in the book title list since faker-ruby#484 (Dec 2015),
when the list was bulk-replaced with titles sourced from Wikipedia.
Both Reginald Hill's novel and Dickens' A Tale of Two Cities Book 1
share the name, so the source list had two entries that were pasted
without deduplication.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

stefannibrasil left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

💯

stefannibrasil merged commit dfb0e67 into faker-ruby:main Apr 14, 2026
9 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL