Skip to content

refactor: add braces to single-line if statements in cli args#443

Merged
sonukapoor merged 1 commit into
OWASP:mainfrom
Kushaal-k:refactor-add-if-braces
May 25, 2026
Merged

refactor: add braces to single-line if statements in cli args#443
sonukapoor merged 1 commit into
OWASP:mainfrom
Kushaal-k:refactor-add-if-braces

Conversation

@Kushaal-k
Copy link
Copy Markdown
Contributor

Closes #435

Summary

Added braces to single-line if statements in src/cli/args.ts to match the project's formatting conventions.

What changed

  • Added braces to single-line if statements
  • Moved conditional bodies onto separate lines
  • Kept all logic and behavior unchanged

Validation

  • Ran npm run build
  • Ran npm test
  • Confirmed all tests still pass

Notes

This is a pure formatting refactor with no intended behavior changes.

Copy link
Copy Markdown
Collaborator

@sonukapoor sonukapoor left a comment

Choose a reason for hiding this comment

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

Mechanical but needed - all the single-line if statements in args.ts now have braces and the logic is unchanged throughout.

One thing before this can merge: the branch is behind main. Could you rebase against main and force-push?

@Kushaal-k Kushaal-k force-pushed the refactor-add-if-braces branch from 4086542 to 4380d19 Compare May 25, 2026 15:15
@sonukapoor sonukapoor merged commit 95cdadf into OWASP:main May 25, 2026
6 checks passed
@sonukapoor
Copy link
Copy Markdown
Collaborator

Thank you for your contribution @Kushaal-k. 💯

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.

refactor: add braces to single-line if statements in src/cli/args.ts

2 participants