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

Need: Lint package.json main property value #111

Open
shellscape opened this issue Apr 19, 2018 · 4 comments
Open

Need: Lint package.json main property value #111

shellscape opened this issue Apr 19, 2018 · 4 comments

Comments

@shellscape
Copy link
Contributor

As a sanity check, we should lint the value for the main property in package.json to ensure it exists (unless dist is specified) if package.json is changed, and definitely lint it as part of the prepublish process.

@alexander-akait
Copy link
Member

@shellscape
Copy link
Contributor Author

Yeah that might be a good option

@alexander-akait
Copy link
Member

@michael-ciniawsky we really need setup this to catch problems with package.json

@michael-ciniawsky
Copy link
Member

michael-ciniawsky commented Aug 17, 2018

Really ? 🙃 It's very rare the main field changes imho and it will also be catched immediately. How should it work in particular via tooling ? But let's leave it open then and discuss :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants