[markdown-remark] Remove unused dependencies

[delucis]

Changes

Follow up to #5785. I missed that acorn and hast-util-to-html are no longer used by @astrojs/markdown-remark and can be removed as dependencies.

Does this need a patch changeset separate to #5785 or is it minimal enough to ignore?

Testing

Existing tests should pass.

Docs

No docs needed.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 5c5adfa

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

[bluwy]

LGTM! Looks like the lockfile needs another update before merging

[delucis]

Hmm, merging seems to have cause a lot more changes in the lockfile vs main. Maybe a bad merge? Yeah, looks like this is undoing lots of the most recent update lockfile CI.

[delucis]

I tried removing the lock file and running pnpm install --lockfile-only like the lockfile CI task does but that doesn’t seem to produce any changes. Not sure what the simple way to resolve this is. Maybe it doesn’t matter?

[bluwy]

Looking into the lockfile, it seems to be reverting the changes of the recent lockfile update. I usually do a merge, and during conflicts, run pnpm i as pnpm supports ootb conflict resolve. Maybe it's worth resetting the branch to main, remove the remark deps, and run pnpm i again to fix it 🤔

[delucis]

This was a good tip. Checked out main, ran pnpm i, checked out my branch, removed the lockfile, ran pnpm i again and it generated the expected lockfile 👍

I actually didn’t see any conflicts while merging, maybe because I have pnpm-merge-driver configured as suggested in CONTRIBUTING.md? Anyway, should be good now!

[bluwy]

Yeah I never quite understood why pnpm-merge-driver was recommended 😅 Glad that's fixed now. PR looks good to me!