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

No package.json #3

Closed
svenefftinge opened this issue Jan 3, 2018 · 6 comments
Closed

No package.json #3

svenefftinge opened this issue Jan 3, 2018 · 6 comments

Comments

@svenefftinge
Copy link
Contributor

There is no package.json.

@brendandburns
Copy link
Contributor

There's a package.json here:

https://github.com/kubernetes-client/javascript/blob/master/node/package.json

I don't think we need a top-level one, do we?

@svenefftinge
Copy link
Contributor Author

ok, it's a little uncommon, though. so you might see more people asking about it.

@brendandburns
Copy link
Contributor

The motivation was because I want both a lib and an examples directory and I didn't want to have my examples inside my lib. How do other projects handle that kind of thing?

Thanks!

@svenefftinge
Copy link
Contributor Author

It would make sense to me if there were different package.json for the code and the examples, but I only see one package.json. You can specify in package.json that examples should not be published.

But the confusion for me might have been caused by the ts files sitting in the same directory as the package.json. They usually sit in some sort of source folder, so you can separate them from .js and .d.ts files. And the package.json in the parent.

@brendandburns
Copy link
Contributor

brendandburns commented Jan 8, 2018 via email

svenefftinge added a commit to svenefftinge/javascript that referenced this issue Jan 8, 2018
…nt#3)

 - renamed ‘node’ to ‘node-client’ to better match the package name
 - moved source code to src folder
 - removed some unneeded libs from package.json
 - fixed some imports
 - updated .gitignore
@svenefftinge
Copy link
Contributor Author

It was easier to just make a proposal, than explaining it :)
#4

Feel free to take anything you find useful and please ask if you have questions.

brendandburns added a commit that referenced this issue Jan 15, 2018
Some improvements for the overall project layout (see #3)
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

No branches or pull requests

2 participants