Skip to content

Condense 3-line if statement to single line using && operator#583

Merged
mmathieum merged 2 commits into
mm/skip_no_current_users_instead_or_errorfrom
copilot/sub-pr-582
Feb 19, 2026
Merged

Condense 3-line if statement to single line using && operator#583
mmathieum merged 2 commits into
mm/skip_no_current_users_instead_or_errorfrom
copilot/sub-pr-582

Conversation

Copilot AI commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

Addresses review feedback from PR #582 to refactor a multi-line conditional exit into a more concise single-line statement.

Changes:

  • Replaced 3-line if/exit block with && operator in shared/app-android/publish_to_current_users.sh

Before:

if [[ ${RESULT} != 0 ]]; then
    exit ${RESULT};
fi

After:

[[ ${RESULT} != 0 ]] && exit ${RESULT};

The logic and behavior remain identical - exit with the result code when non-zero.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor script logic to condense multiple lines into one Condense 3-line if statement to single line using && operator Feb 19, 2026
Copilot AI requested a review from mmathieum February 19, 2026 13:55
@mmathieum mmathieum marked this pull request as ready for review February 19, 2026 14:10
@mmathieum mmathieum merged commit c2c47ac into mm/skip_no_current_users_instead_or_error Feb 19, 2026
@mmathieum mmathieum deleted the copilot/sub-pr-582 branch February 19, 2026 14:10
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