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

add vite-plugin-require-transform #308

Closed
wants to merge 2 commits into from

Conversation

WarrenJones
Copy link
Contributor

@WarrenJones WarrenJones commented Nov 3, 2021

add vite-plugin-require-transform under Transformers

Please make sure all the requirements are satisfied, otherwise the PR could be closed without further notice.

Checklist

  • Title as described.
  • Make sure you put things in the right category.
  • The description of your item should be a sentence with less than 24 words.
  • Avoid using links and parentheses in description.
  • Omit unnecessary words already provided in the context (e.g. for Vite, a Vite plugin).
  • Use proper case for terms (e.g. GitHub, TypeScript, ESLint, etc.)
  • When mentioning tools, omit versions whenever possible (e.g. use TypeScript instead of TypeScript 4.x, but keep Vue 2 and Vue 3 as they're incompatible).
  • When mentioning package names, use quotes whenever possible.
  • Always add your items to the end of a list.

Plugins/Tools

  • The plugin/tool is working with Vite 2.x and onward.
  • The project is Open Source.
  • The project follows the Vite Plugins Conventions.
  • The plugin uses Vite-specific hooks and can't be implemented as a Compatible Rollup Plugin.
  • The repo should be at least 30 days old.
  • The documentation is in English.
  • The project is active and maintained (inactive projects for longer 6 months will be removed without further notice).
  • The project accepts contributions.
  • Not a commercial product.

Starter Templates

  • The starter template is working with Vite 2.x and onward.
  • The documentation is in English.
  • The starter template most provide enough instructions / documentation about how to start up and what's is included.
  • A screenshot or a live demo should be included.
  • The repo should be at least 30 days old.
  • Should be differentiable from the existing starter templates.

Apps/Websites

  • The website is available without errors and load in a reasonable amount of time.
  • The website must be Open Source and showing it's using Vite intensively.
  • The website is original and not too simple. For that reason, blogs and simple landing pages are rejected.

add vite-plugin-require-transform under Transformers
Copy link
Collaborator

@Scrum Scrum left a comment

Choose a reason for hiding this comment

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

Checklist

  • git: pull request template structure not broken
  • git: pull request template not selected type
  • git: pull request template not checked Checklist
  • git: empty commit
  • git: link not to repository
  • git: link description contains a link to the author / third-party resource
  • git: link description no for Vite or Vite plugin
  • git: link description short description short description (one sentence in 24 words, no expanded explanation)
  • git: terms in the wrong formats (e.g. Typescript over TypeScript)
  • git: broken PR
  • git: no conflicts resolved
  • git: doublicate
  • time: not updated for more than 6 months
  • category: not in its category
  • lists: not the last on the list
  • docs: no documentation
  • docs: non-english documentation
  • project: commercial product
  • project: dead project
  • project: does not accept donat
  • site: no link to the site
  • site: site with errors
  • site: not actively using vuejs
  • site: not original

Follow the rules CONTRIBUTING

@WarrenJones
Copy link
Contributor Author

I already left a description includes "Vite plugin".A Vite plugin that converts the code from require syntax to import

@Scrum
Copy link
Collaborator

Scrum commented Nov 3, 2021

I already left a description includes "Vite plugin".A Vite plugin that converts the code from require syntax to import

no for Vite or Vite plugin

@WarrenJones
Copy link
Contributor Author

A Vite plugin that converts the code from require syntax to import . This not count? where should I put the word,not here?

@WarrenJones
Copy link
Contributor Author

I already left a description includes "Vite plugin".A Vite plugin that converts the code from require syntax to import

no for Vite or Vite plugin
A Vite plugin that converts the code from require syntax to import . This not count? where should I put the word,not here?

@Scrum
Copy link
Collaborator

Scrum commented Nov 3, 2021

A Vite plugin that converts the code from require syntax to import . This not count? where should I put the word,not here?

Read carefully https://github.com/vitejs/awesome-vite/blob/master/.github/contributing.md#contributing-guide

@WarrenJones
Copy link
Contributor Author

A Vite plugin that converts the code from require syntax to import . This not count? where should I put the word,not here?

Read carefully https://github.com/vitejs/awesome-vite/blob/master/.github/contributing.md#contributing-guide

👌 I already removed that words

@WarrenJones WarrenJones closed this Nov 5, 2021
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.

None yet

2 participants