Skip to content

Fix vite-plugin-react-svg for dependencies#59777

Merged
sandersn merged 1 commit intomasterfrom
fix-vite-plugin-react-svg-dependencies
Apr 9, 2022
Merged

Fix vite-plugin-react-svg for dependencies#59777
sandersn merged 1 commit intomasterfrom
fix-vite-plugin-react-svg-dependencies

Conversation

@sandersn
Copy link
Copy Markdown
Contributor

@sandersn sandersn commented Apr 8, 2022

vite-plugin-react-svg's dependencies require

  1. 'lib' to include 'dom'
  2. The minimum typescript version to be 4.0.

vite-plugin-react-svg's dependencies require

1. 'lib' to include 'dom'
2. The minimum typescript version to be 4.0.
@typescript-bot
Copy link
Copy Markdown
Contributor

typescript-bot commented Apr 8, 2022

@sandersn Thank you for submitting this PR!

This is a live comment which I will keep updated.

1 package in this PR

Code Reviews

This PR can be merged once it's reviewed by a DT maintainer.

You can test the changes of this PR in the Playground.

Status

  • ✅ No merge conflicts
  • ✅ Continuous integration tests have passed
  • 🕐 Only a DT maintainer can approve changes without tests

Once every item on this list is checked, I'll ask you for permission to merge and publish the changes.


Diagnostic Information: What the bot saw about this PR
{
  "type": "info",
  "now": "-",
  "pr_number": 59777,
  "author": "sandersn",
  "headCommitOid": "0f2ed66c6d74b618b710786fe58f832ae7c47723",
  "mergeBaseOid": "fa738deff7af38eef4d8aa6a1e066a0277ab9e9e",
  "lastPushDate": "2022-04-08T23:53:27.000Z",
  "lastActivityDate": "2022-04-09T15:43:38.000Z",
  "hasMergeConflict": false,
  "isFirstContribution": false,
  "tooManyFiles": false,
  "hugeChange": false,
  "popularityLevel": "Well-liked by everyone",
  "pkgInfo": [
    {
      "name": "vite-plugin-react-svg",
      "kind": "edit",
      "files": [
        {
          "path": "types/vite-plugin-react-svg/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/vite-plugin-react-svg/tsconfig.json",
          "kind": "package-meta-ok"
        }
      ],
      "owners": [
        "priyanshurav"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Well-liked by everyone"
    }
  ],
  "reviews": [
    {
      "type": "approved",
      "reviewer": "eps1lon",
      "date": "2022-04-09T15:43:38.000Z",
      "isMaintainer": false
    }
  ],
  "mainBotCommentID": 1093494357,
  "ciResult": "pass"
}

@typescript-bot typescript-bot added the Untested Change This PR does not touch tests label Apr 8, 2022
@typescript-bot
Copy link
Copy Markdown
Contributor

🔔 @priyanshurav — please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

@typescript-bot typescript-bot added the Other Approved This PR was reviewed and signed-off by a community member. label Apr 9, 2022
@sandersn sandersn merged commit f472421 into master Apr 9, 2022
@sandersn sandersn deleted the fix-vite-plugin-react-svg-dependencies branch April 9, 2022 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Other Approved This PR was reviewed and signed-off by a community member. Untested Change This PR does not touch tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants