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

Update CustomWrapper.cls #851

Merged
merged 1 commit into from
Sep 22, 2023
Merged

Update CustomWrapper.cls #851

merged 1 commit into from
Sep 22, 2023

Conversation

Damecek
Copy link
Contributor

@Damecek Damecek commented Sep 7, 2023

removed comment about inner classes, which are now supported

What does this PR do?

remove redundant comment

What issues does this PR fix or reference?

clarify false statement

The PR fulfills these requirements:

[x] Tests for the proposed changes have been added/updated.
[x] Code linting and formatting was performed.

removed comment about inner classes, which are now supported
@Damecek Damecek requested a review from a team as a code owner September 7, 2023 12:56
@salesforce-cla
Copy link

salesforce-cla bot commented Sep 7, 2023

Thanks for the contribution! Before we can merge this, we need @Damecek to sign the Salesforce Inc. Contributor License Agreement.

@Damecek
Copy link
Contributor Author

Damecek commented Sep 7, 2023

I've already signed the CLA on 2023-09-07

@pozil
Copy link
Contributor

pozil commented Sep 22, 2023

Thanks for the contribution and good catch.

@pozil pozil merged commit cae86d7 into trailheadapps:main Sep 22, 2023
5 of 6 checks passed
@Damecek Damecek deleted the patch-1 branch September 22, 2023 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants