Skip to content

feat(dev-lib): enable no-nested-ternary oxlint rule - #154

Merged
fiddur merged 2 commits into
mainfrom
lint-no-nested-ternary-enable
Sep 2, 2026
Merged

feat(dev-lib): enable no-nested-ternary oxlint rule#154
fiddur merged 2 commits into
mainfrom
lint-no-nested-ternary-enable

Conversation

@fiddur

@fiddur fiddur commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Turns on the no-nested-ternary oxlint rule in the shared config.

Stacked on #153, which clears the existing violations here; GitHub will retarget this to main once that merges. Consumers should be clean before picking up the new dev-lib version.

fiddur and others added 2 commits September 1, 2026 15:40
Rewrites the 8 nested ternary expressions across js-lib, nodejs-lib,
backend-lib and datastore-lib as early returns or if/else. Pure
readability change, behaviour is identical.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DXP6cFK7Mu2Jdh487AXmTc
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DXP6cFK7Mu2Jdh487AXmTc
Base automatically changed from lint-no-nested-ternary to main September 1, 2026 15:01
@fiddur
fiddur marked this pull request as ready for review September 2, 2026 13:40
@fiddur
fiddur merged commit 15793ab into main Sep 2, 2026
3 checks passed
@fiddur
fiddur deleted the lint-no-nested-ternary-enable branch September 2, 2026 13:41
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