Skip to content

Conversation

@WillieRuemmele
Copy link
Contributor

What does this PR do?

if you project delete source on something that's forceignored, you can have a bad time
this makes you have less of a bad time, prints a warning

What issues does this PR fix or reference?

@W-17013213@

@shetzel
Copy link
Contributor

shetzel commented Jan 10, 2025

QA
Try to delete an LWC tests dir. With the changes you see a warning like this:

Warning: Attempting to delete metadata that conflicts with a .forceignore entry: .../force-app/main/default/lwc/brokerCard/__tests__/brokerCard.test.js,.../force-app/main/default/lwc/brokerCard/__tests__/data
Update the .forceignore file and try again.
? Are you sure you want to proceed? (y/N)

@shetzel shetzel merged commit 1587393 into main Jan 10, 2025
43 checks passed
@shetzel shetzel deleted the wr/sourceDeleteUndefined branch January 10, 2025 22:28
@iowillhoit iowillhoit changed the title fix: add warning when nothing to delete W-17013213 fix: add warning when nothing to delete Jan 27, 2025
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