Skip to content

Refactor: Switch to let & const from var in JS, JSX, TS & TSX code-blocks#3755

Merged
cortinico merged 1 commit into
react:mainfrom
Pranav-yadav:Pranav-yadav/var-to-let-const
Jun 12, 2023
Merged

Refactor: Switch to let & const from var in JS, JSX, TS & TSX code-blocks#3755
cortinico merged 1 commit into
react:mainfrom
Pranav-yadav:Pranav-yadav/var-to-let-const

Conversation

@Pranav-yadav

@Pranav-yadav Pranav-yadav commented Jun 12, 2023

Copy link
Copy Markdown
Contributor

Summary

This is diff update the number of JS, JSX, TS & TSX code-blocks to switch to let & const from var.
And of course some markdown formatting touchups like code formatting.

P.S.: I came across these var references when working on #3732 😇

@netlify

netlify Bot commented Jun 12, 2023

Copy link
Copy Markdown

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit d8c0038
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/648739569aed2a000896c19a
😎 Deploy Preview https://deploy-preview-3755--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@Pranav-yadav Pranav-yadav force-pushed the Pranav-yadav/var-to-let-const branch from 87cff72 to d8c0038 Compare June 12, 2023 15:27
@Pranav-yadav Pranav-yadav changed the title Refactor: Switch to let & const from var in JS, JSX, TS & TSX files Refactor: Switch to let & const from var in JS, JSX, TS & TSX code-blocks Jun 12, 2023

@Simek Simek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thank you for those fixes, LGTM! 👍

@cortinico cortinico merged commit f9b04b4 into react:main Jun 12, 2023
@Pranav-yadav Pranav-yadav deleted the Pranav-yadav/var-to-let-const branch June 12, 2023 19:15
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.

4 participants