Skip to content

Conversation

@sf-aastha-paruthi
Copy link
Collaborator

What does this PR do?

If folder name contains reserved names like lwc, lables, others the deployment fails.
Added validation to check the name and inform the user.

What issues does this PR fix or reference?

@sf-aastha-paruthi sf-aastha-paruthi requested a review from a team as a code owner January 16, 2026 05:42
Copy link

@shaurabh-tiwari-git shaurabh-tiwari-git left a comment

Choose a reason for hiding this comment

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

Test with folderName /classes as well for apex before merging. Other changes look good.

@sf-aastha-paruthi
Copy link
Collaborator Author

Changed the folder name to classes and tried the unmanaged package deployment.
It succeeded.
image

@sf-aastha-paruthi sf-aastha-paruthi merged commit d1dd087 into salesforcecli:release260omnistudiopackage Jan 16, 2026
1 check passed
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