Skip to content
New issue

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

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

Already on GitHub? # to your account

Null safe access #185

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

pankratz76
Copy link

hello guys can you please give me a kickstart how to do it ? thanks

@timtebeek timtebeek requested a review from arodionov January 3, 2025 16:05
@timtebeek timtebeek marked this pull request as draft January 3, 2025 16:05
@timtebeek timtebeek changed the title null save access Null safe access Jan 3, 2025
@arodionov
Copy link
Contributor

Hi @punkratz312
What is the intention of your PR?
What would you like to start with?

Comment on lines +45 to +47
`),
//language=typescript
typeScript(`
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure about the specifics for typeScript here, but I would expect a method call with two arguments, as opposed to two method calls.

Suggested change
`),
//language=typescript
typeScript(`
`,`

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

4 participants