search

justforuse / vue-mathjax-next

Vue3 + Mathjax plugin
https://vue-mathjax-next.vercel.app/
29 stars 2 forks source link

Add dev dependency, exclude lib/ from SCM, remove console.log #1

Closed eagleoflqj closed 2 years ago

justforuse commented 2 years ago

@eagleoflqj Why remove lib/? I think that is the file that referred in package.json file

eagleoflqj commented 2 years ago

Automatically generated files (except package-lock.json) should be excluded from source code management.

justforuse commented 2 years ago

@eagleoflqj Thanks your job! I spent few days to check how the well-known packages do this staff by jsdelivr, I misunderstood first😅, so you mean that the lib files should not be push to github, it should published by owner?

eagleoflqj commented 2 years ago

so you mean that the lib files should not be push to github, it should published by owner?

Exactly! I do see repos like vueuc that excludes lib/ from git but has "main": "lib/index.js" in package.json