docs(pages/v2): remove default values from serverBuildTarget section

[MichaelDeBoey]

Follow-up of #5934

Since we're talking about replacement, I figured that we don't need to specify the default values in here in order to make it work as expected

[changeset-bot]

⚠️ No Changeset found

Latest commit: 1f71fcd

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.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

[brophdawg11]

I think the default values can be helpful in this context so you know the full extent of what the old thing was doing for you. The main docs on remix.config.js do include the defaults so folks can figure out if they need all these fields if they want to?

[MichaelDeBoey]

@brophdawg11 What about adding them as comments instead then?
I just don't want people to think they need to include all these values in order to make it work.

[brophdawg11]

Something like this?

serverModuleFormat: "cjs", // default value, can be removed

@jacob-ebey what do you think?

[jacob-ebey]

@MichaelDeBoey, let's leave the values in the config but add the comment next to them for now like @brophdawg11 suggested.

[MichaelDeBoey]

@brophdawg11 @jacob-ebey I've updated this PR as requested 👍