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

chore: fix forceignore warnings on commented lines #1462

Merged
merged 1 commit into from
Nov 26, 2024
Merged

Conversation

WillieRuemmele
Copy link
Member

@WillieRuemmele WillieRuemmele commented Nov 26, 2024

What does this PR do?

changes forceignore warnings to ignore commented lines

What issues does this PR fix or reference?

[skip-validate-pr]

forcedotcom/cli#3127

Functionality Before

would warn on
# force-app\main

Functionality After

will not warn on example above

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@WillieRuemmele WillieRuemmele requested a review from a team as a code owner November 26, 2024 21:04
@soridalac
Copy link

QA Notes

✅ : ignore warnings backslash with comment lines

@soridalac soridalac merged commit 811b971 into main Nov 26, 2024
51 of 52 checks passed
@soridalac soridalac deleted the wr/forceIgnore branch November 26, 2024 22:57
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.

None yet

3 participants