Skip to content

Conversation

arcanis
Copy link
Member

@arcanis arcanis commented Oct 16, 2023

What's the problem this PR addresses?

One of our tests on the @types auto-add was testing @iarna/toml, but this package actually contains types (but was reported as not having them due to a bug on Algolia's side).

How did you fix it?

I changed the package to @babel/parser, which doesn't have builtin types.

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@arcanis arcanis merged commit 3e2dfb2 into master Oct 16, 2023
@arcanis arcanis deleted the mael/iarna-test branch October 16, 2023 10:21
merceyz pushed a commit that referenced this pull request Oct 21, 2023
**What's the problem this PR addresses?**

One of our tests on the `@types` auto-add was testing `@iarna/toml`, but
this package actually contains types (but was reported as not having
them due to a bug on Algolia's side).

**How did you fix it?**

I changed the package to `@babel/parser`, which doesn't have builtin
types.

**Checklist**
<!--- Don't worry if you miss something, chores are automatically
tested. -->
<!--- This checklist exists to help you remember doing the chores when
you submit a PR. -->
<!--- Put an `x` in all the boxes that apply. -->
- [x] I have read the [Contributing
Guide](https://yarnpkg.com/advanced/contributing).

<!-- See
https://yarnpkg.com/advanced/contributing#preparing-your-pr-to-be-released
for more details. -->
<!-- Check with `yarn version check` and fix with `yarn version check
-i` -->
- [x] I have set the packages that need to be released for my changes to
be effective.

<!-- The "Testing chores" workflow validates that your PR follows our
guidelines. -->
<!-- If it doesn't pass, click on it to see details as to what your PR
might be missing. -->
- [x] I will check that all automated PR checks pass before the PR gets
reviewed.

(cherry picked from commit 3e2dfb2)
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.

2 participants