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

doc: update var to const in all Readmes #1024

Merged
merged 1 commit into from
Aug 7, 2021

Conversation

jonchurch
Copy link
Member

@jonchurch jonchurch commented Aug 6, 2021

Update var to const in Readme and all translations. This was done with a search replace on each document, I believe but did not verify that all code examples are the same in all translations.

Interestingly though, I think the English readme might have an extra codeblock that the others don't. Someone should check the blame and see if we need to update the translations with an example. Although I guess we would then need to translate that example haha.

Edit: Found it and it's of course something I reviewed. Looking back at it, I'm not sure that example was necessary. I think it just repeats the example above it. Maybe the elaborated explanation should get moved up.

closes #1012

@jonchurch jonchurch added the docs label Aug 6, 2021
Copy link
Member

@LinusU LinusU left a comment

Choose a reason for hiding this comment

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

Awesome 👏

@LinusU LinusU merged commit a5b7d5e into expressjs:master Aug 7, 2021
@LinusU
Copy link
Member

LinusU commented Aug 7, 2021

Looking back at it, I'm not sure that example was necessary. I think it just repeats the example above it. Maybe the elaborated explanation should get moved up.

Yeah, I think you are right, I'm open to changing it 👍

nevilm-lt pushed a commit to nevilm-lt/multer that referenced this pull request Apr 21, 2022
nevilm-lt pushed a commit to nevilm-lt/multer that referenced this pull request Apr 22, 2022
nevilm-lt pushed a commit to nevilm-lt/multer that referenced this pull request Apr 22, 2022
himanshiLt pushed a commit to himanshiLt/multer that referenced this pull request Apr 26, 2022
himanshiLt pushed a commit to himanshiLt/multer that referenced this pull request Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[docs] [question] Why does the readme use var in place of let / const ?
2 participants