Skip to content

fix: accept 'passed' as alias for 'pass' in FlickreviewR status#80

Merged
DaxServer merged 2 commits intomainfrom
fix/flickr-passed-status-alias
Apr 3, 2026
Merged

fix: accept 'passed' as alias for 'pass' in FlickreviewR status#80
DaxServer merged 2 commits intomainfrom
fix/flickr-passed-status-alias

Conversation

@DaxServer
Copy link
Copy Markdown
Owner

The FlickreviewR template supports status aliases (Template:FlickreviewR/status_aliases).
'passed' is a valid alias for 'pass', but the bot was incorrectly skipping files with this status.

Also updates CLAUDE.md to document the alias system.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation and logic for the FlickreviewR template to support status aliases, specifically allowing 'passed' as an alternative to 'pass'. A suggestion was made to update the warning log message to reflect this change for consistency between the code logic and the output logs.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@DaxServer DaxServer merged commit 35ef94c into main Apr 3, 2026
3 checks passed
@DaxServer DaxServer deleted the fix/flickr-passed-status-alias branch April 3, 2026 16:26
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.

1 participant