Skip to content

Conversation

@ryan953
Copy link
Member

@ryan953 ryan953 commented Mar 21, 2025

No description provided.

@ryan953 ryan953 requested review from a team as code owners March 21, 2025 23:56
@ryan953 ryan953 requested review from a team March 21, 2025 23:56
@ryan953 ryan953 requested a review from a team as a code owner March 21, 2025 23:56
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 21, 2025
@ryan953 ryan953 changed the title chore(eslint): Enable and auto-fix unicorn/prefer-includes chore(eslint): Enable and mostly auto-fix unicorn/prefer-includes Mar 22, 2025
Comment on lines +255 to +259
const priorityTags = isMobilePlatform(project.platform)
? MOBILE_TAGS
: frontend.some(val => val === project?.platform)
: frontend.includes(project.platform ?? 'other')
? FRONTEND_TAGS
: backend.some(val => val === project?.platform)
: backend.includes(project.platform ?? 'other')
Copy link
Member Author

Choose a reason for hiding this comment

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

this needed some manual fixing up. I added ?? 'other' in there.

Comment on lines +302 to +306
isMobilePlatform(project.platform)
? MOBILE_TAGS
: frontend.some(val => val === project?.platform)
: frontend.includes(project.platform ?? 'other')
? FRONTEND_TAGS
: backend.some(val => val === project?.platform)
: backend.includes(project.platform ?? 'other')
Copy link
Member Author

Choose a reason for hiding this comment

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

this needed some manual fixing up. I added ?? 'other' in there.

@ryan953 ryan953 merged commit db83f5b into master Mar 24, 2025
40 checks passed
@ryan953 ryan953 deleted the ryan953/prefer-includes branch March 24, 2025 18:03
@github-actions github-actions bot locked and limited conversation to collaborators Apr 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants