docs: Modified variable names for better clarity #8687

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.

Sign up for GitHub

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

Jump to bottom

Closed

IgnorantSapient wants to merge 1 commit into sveltejs:master from IgnorantSapient:edit-reduce-var-name

Contributor

IgnorantSapient commented Jun 2, 2023 •

edited

Loading

Variable names t and n would be confusing for beginners, hence changing it to total and currentNumber

Prefix your PR title with feat:, fix:, chore:, or docs:.
This message body should clearly illustrate what problems it solves.

npm run test - tests are passing:


          Modified variable names for better clarity

e3eedf2

vercel bot commented Jun 2, 2023

@IgnorantSapient is attempting to deploy a commit to the Svelte Team on Vercel.

A member of the Team first needs to authorize it.

Member

benmccann commented Jun 2, 2023

The tutorial is moving here: https://learn.svelte.dev/tutorial/updating-arrays-and-objects

If you want to send a PR, it'd be better to send it to that version: https://github.com/sveltejs/learn.svelte.dev/tree/main/content/tutorial/01-svelte/02-reactivity/04-updating-arrays-and-objects

IgnorantSapient closed this

Contributor Author

IgnorantSapient commented Jun 6, 2023

Thanks for the reply! @benmccann, here we go:

sveltejs/learn.svelte.dev#400 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet