Skip to content

docs: fix MarkDuplicates option name#2051

Open
titanniya542-spec wants to merge 1 commit into
broadinstitute:masterfrom
titanniya542-spec:fix/markduplicates-remove-duplicates-doc
Open

docs: fix MarkDuplicates option name#2051
titanniya542-spec wants to merge 1 commit into
broadinstitute:masterfrom
titanniya542-spec:fix/markduplicates-remove-duplicates-doc

Conversation

@titanniya542-spec

Copy link
Copy Markdown

Description

This fixes the MarkDuplicates documentation string so it refers to the actual REMOVE_DUPLICATES option rather than REMOVE_DUPLICATE.

Fixes #2042.


Checklist (never delete this)

Never delete this, it is our record that procedure was followed. If you find that for whatever reason one of the checklist points doesn't apply to your PR, you can leave it unchecked but please add an explanation below.

Content

  • Added or modified tests to cover changes and any new functionality
  • Edited the README / documentation (if applicable)
  • All tests passing on github actions

Review

  • Final thumbs-up from reviewer
  • Rebase, squash and reword as applicable

Tests were not added because this is a one-word documentation string correction. I verified locally with git diff --check and rg that the documented option name now matches REMOVE_DUPLICATES.

AI disclosure: I used OpenAI Codex to help prepare this change.

@yfarjoun yfarjoun 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.

praise
Thanks!

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.

Documentation spelling mistake REMOVE_DUPLICATE -> REMOVE_DUPLICATES

2 participants