docs(setting-up-your-local-dev-environment): beautify some parts

[muescha]

Description

• restructure some parts to not miss there a 2 different gatsby-cli and gatsby-dev-cli
• some headings
  - add a hint about Pull Bot removed by request

Related Issues

n/a

Wordings

i am not good in wording docs, suggestions are welcome

[muescha]

i feel the yarn run watch should be under the feature changes heading and not in docs changes heading?

[muescha]

should we also add some notes about develop-runner - i only found this description here #19092 (review)

This is so cool! 👍

I think you are missing child/parent relationship cases, so when child/parent relationship is added either through action or by setting 'parent' field.

We usually test against .org (www folder in monorepo) using gatsby-dev-cli. You can test against multiple sites using develop-runner. You need to modify source code a bit, but it's still very handy.

We often publish a pre-release version for packages with big changes, then you can use it to test with develop-runner. Eg you can reuse tag @schema-customization. (To do it, go to gatsby/packages/gatsby and do yarn publish --tag schema-customization).

[pieh]

should we also add some notes about develop-runner - i only found this description here #19092 (review)

I'm not sure - we ourselves use this pretty rarely, only when there is rewrite of one of major core function that potentially could introduce bugs (or unforeseen breaking changes). Last time we used it (before schema rebuilding PR) was materialization for data layer.

The script itself also takes a loong time to run, so I don't want to force it on contributors, as it's not needed for most of PRs.

It makes sense to me to make a note about it somewhere, but this could be separate PR to not scope creep this one?

[pieh]

i feel the yarn run watch should be under the feature changes heading and not in docs changes heading?

Yes, if none of the packages is changed - we don't need yarn watch - so purely documentation work, potentially only need to install deps for www and run gatsby develop there (but it also isn't strictly needed)

[muescha]

So I can leaveThe watch part it as is?

[muescha]

placed suggestion about developer-runner into #19380

[laurieontech]

This is looking really good. Love the additional details for such a popular write up.

[muescha]

updated from suggestions of @laurieontech and @jbampton

[jbampton]

Looks better thanks 👍

[muescha]

:( CI test are failing - but there are out of my scope

[jbampton]

Also maybe you should update the PR description to remove add a hint about Pull Bot

[muescha]

done