Skip to content

Conversation

bluwy
Copy link
Contributor

@bluwy bluwy commented Aug 27, 2024

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and couldn’t find anything (or linked relevant results below)
  • If applicable, I’ve added docs and tests

Description of changes

Remove hast-util-raw as it's not used in the published files. It's only used in test/raw.js for ambient types, but I found from https://github.com/syntax-tree/hast-util-raw?tab=readme-ov-file#types and https://github.com/syntax-tree/mdast-util-to-hast?tab=readme-ov-file#types that the types should be sourced from mdast-util-to-hast, so I switched to that instead and hast-util-raw is no longer used anywhere else.

@github-actions github-actions bot added the 👋 phase/new Post is being triaged automatically label Aug 27, 2024

This comment has been minimized.

@github-actions github-actions bot added 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Aug 27, 2024
@wooorm
Copy link
Member

wooorm commented Aug 27, 2024

There wasn’t really a place where “raw” was coming from, so it used to be defined in both places.
That’s now cleaned, as you noticed from the docs, they’re in mdast-util-to-hast.

@wooorm wooorm changed the title Remove unused hast-util-raw dependency Remove unused dependency Aug 27, 2024
@wooorm wooorm merged commit 0a30b45 into syntax-tree:main Aug 27, 2024
4 checks passed
@wooorm wooorm added the 💪 phase/solved Post is done label Aug 27, 2024

This comment has been minimized.

@github-actions github-actions bot removed the 🤞 phase/open Post is being triaged manually label Aug 27, 2024
@wooorm
Copy link
Member

wooorm commented Aug 27, 2024

9.0.2, thanks!

@bluwy bluwy deleted the remove-hast-util-raw branch August 27, 2024 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💪 phase/solved Post is done
Development

Successfully merging this pull request may close these issues.

2 participants