Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[README] Update link to excerpter pkg #6318

Merged
merged 1 commit into from
Jan 17, 2025
Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jan 9, 2025

Fixes 4040 of link into site-shared repo.

@chalin
Copy link
Contributor Author

chalin commented Jan 9, 2025

Hi @parlough. On a related note, I thought that you had JS (more specifically NPM-package usable) versions of the code excerpter and code-excerpt syncing tools. Did I just imagine that or are there such versions of the tools?

Copy link
Contributor

@MaryaBelanger MaryaBelanger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @chalin! Merging in the fix but @parlough please note there's a question for you in the PR

@MaryaBelanger MaryaBelanger merged commit 66126ca into dart-lang:main Jan 17, 2025
8 of 10 checks passed
@parlough
Copy link
Member

Hi @parlough. On a related note, I thought that you had JS (more specifically NPM-package usable) versions of the code excerpter and code-excerpt syncing tools. Did I just imagine that or are there such versions of the tools?

Hey 👋! Thanks for fixing this link!

Unfortunately we don't build JS compiled versions of the tools or publish them, as we just use them through Dart-based tooling on the site (/tool/dart_site/lib/src/commands/refresh_excerpts.dart).

Do you have a use case in mind? If the tool is useful for you, I could maybe look into polishing the packages and publishing them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants