vue-tagsinput icon indicating copy to clipboard operation
vue-tagsinput copied to clipboard

Major/move to typescript vuecli

Open seriouslag opened this issue 3 years ago • 4 comments

This PR is the start of migrating to typescript, exporting type definitions, moving to use vue cli as the bundler for the library, using vue cli as the dev server.

  • Migrated all js to typescript
  • fixed bugs found when using typescript
  • changed package.json main field to commonjs to allow for easier importing for library consumers.
  • changed to use new babel.config.js file
  • removed need to commit bulma styles, will be installed from package.json for dev work
  • Changed default export to a Vue plugin install function
  • Named exporting the needed types and component

Note dist files will need to be created by a CI pipeline since I removed them from the repo

seriouslag avatar Sep 14 '21 17:09 seriouslag

@AlexMordred would you care to take a look at this PR :)

seriouslag avatar Sep 20 '21 01:09 seriouslag

@seriouslag in https://github.com/voerro/vue-tagsinput/issues/132#issuecomment-808876122, Alex says:

Unfortunately, I am not familiar with TypeScript as of now

So, I think we must follow some checklist, like so:

  • it working without changes after switching to PR branch and installing dependencies
  • it have no type errors
  • ...and so on

Grawl avatar Sep 23 '21 01:09 Grawl

@Grawl

Would you care to add more?

Are you experiencing any issues when you pull the branch and install deps?

I am not seeing any issues on my end. I published the changes to a package to use on my project until these get moved.

https://www.npmjs.com/package/@seriouslag/vue-tagsinput

seriouslag avatar Sep 23 '21 18:09 seriouslag

Just tested it and it works

Grawl avatar Sep 24 '21 03:09 Grawl