Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: concise flag on status to omit ignored files #382

Merged
merged 3 commits into from
Jan 12, 2022

Conversation

mshanemc
Copy link
Contributor

What does this PR do?

use built-in "concise" flag for source:beta:status to omit ignored files (rather than a new flag)

What issues does this PR fix or reference?

@W-10271553@
forcedotcom/cli#1312

Co-authored-by: Juliet Shackell <[email protected]>
@WillieRuemmele
Copy link
Contributor

local change to apex class

 ➜  sfdx force:source:beta:status
Source Status
STATE          FULL NAME  TYPE       PROJECT PATH
─────────────  ─────────  ─────────  ───────────────────────────────────────
Local Changed  post       ApexClass  force-app/main/default/classes/post.cls
Remote Add     Admin      Profile

✅ : shows as ignored when added to .forceignore force-app/main/default/classes/post.*

➜  dreamhouse-lwc git:(main) ✗  hub:(DevHub) scratch:([email protected])
 ➜  sfdx force:source:beta:status
Source Status
IGNORED  STATE          FULL NAME  TYPE       PROJECT PATH
───────  ─────────────  ─────────  ─────────  ───────────────────────────────────────
true     Local Changed  post       ApexClass  force-app/main/default/classes/post.cls
         Remote Add     Admin      Profile

✅ : removed from output when using the --concise flag

➜  dreamhouse-lwc git:(main) ✗  hub:(DevHub) scratch:([email protected])
 ➜  sfdx force:source:beta:status --concise
Source Status
STATE       FULL NAME  TYPE     PROJECT PATH
──────────  ─────────  ───────  ────────────
Remote Add  Admin      Profile

✅ : DOESN'T change --json output

@WillieRuemmele WillieRuemmele merged commit 2dcbac8 into main Jan 12, 2022
@WillieRuemmele WillieRuemmele deleted the sm/concise-status branch January 12, 2022 18:47
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.

3 participants