docs: update message about this when using regular function

[iamandrewluca]

An arrow function cannot change the this context.

In the example before, this was always the global object (global in node, window in browser)

[changeset-bot]

⚠️ No Changeset found

Latest commit: a2f1863

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[PuruVJ]

Thanks Andrew for the PR!

You don't this in the case above, instead you use ctx as provided as the first argument

[iamandrewluca]

I thought the comment was misleading with an arrow function 🤔
In any case, if that was the purpose please close the PR 😄

// `this` is `ctx` in modern build

[PuruVJ]

Actually yeah, it is kind of confusing. Could you tweak the text to mean "ctx instead of this) or similar? Happy to merge that!

[pkg-pr-new]

Run & review this pull request in StackBlitz Codeflow.

commit: 0e7a309

neotraverse

npm i https://pkg.pr.new/PuruVJ/neotraverse@7

---

templates

• default

[iamandrewluca]

@PuruVJ made some changes

[PuruVJ]

Thanks!