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

updated import to new location for yaml/_loader/loader.ts #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JacobWeisenburger
Copy link

@iuioiua
Copy link

iuioiua commented Mar 17, 2023

This is what should be used as it's part of the public API, instead of the currently used internal code:

import { parse } from "https://deno.land/[email protected]/yaml/parse.ts";

As stated in the README for the Deno Standard Library:

Don't link to / import any module whose path:

  • Has a name or parent with an underscore prefix: _foo.ts, _util/bar.ts.
    ...

These elements are not considered part of the public API, thus no stability is guaranteed for them.

@nberlette
Copy link

@ubersl0th Is there anything holding this back from being updated to the current stdlib version and merged into main? It's currently preventing your module from being used in any capacity.

Thanks and Happy Easter,
Nick

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