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

Publish to npm? #71

Open
1EDExg0ffyXfTEqdIUAYNZGnCeajIxMWd2vaQeP opened this issue Apr 17, 2023 · 16 comments · Fixed by #99
Open

Publish to npm? #71

1EDExg0ffyXfTEqdIUAYNZGnCeajIxMWd2vaQeP opened this issue Apr 17, 2023 · 16 comments · Fixed by #99
Assignees
Labels
enhancement New feature or request

Comments

@1EDExg0ffyXfTEqdIUAYNZGnCeajIxMWd2vaQeP

Would be simpler to use and update to newer versions.

@melloware
Copy link
Member

Also has been mentioned on Discord and PrimeTek has put it on their list!

@melloware melloware added documentation Improvements or additions to documentation enhancement New feature or request and removed documentation Improvements or additions to documentation labels Apr 17, 2023
@C0ZEN
Copy link

C0ZEN commented Jul 11, 2023

I am so surprised that this is not the case already 🤔

@Anders142536
Copy link
Contributor

I tried to add this repository to our dependencies via github dependency syntax and the install fails, probably because there is no package.json file. Maybe just add that? I don't think it has to be hosted on npm necessarily.

@melloware
Copy link
Member

melloware commented Aug 2, 2023

Do you have instructions on how your included using GitHub syntax and then I can test and add a package.json. Thanks!
I didn't know that was possible!

@Anders142536
Copy link
Contributor

Oh of course, I should have included a link to the documentation in the first message already:
https://docs.npmjs.com/cli/v7/configuring-npm/package-json#github-urls

@melloware
Copy link
Member

OK I added package.json but didn't test it yet if you want to test it: https://github.com/primefaces/primelocale/blob/main/package.json

@Anders142536
Copy link
Contributor

npm definitely treats this now as an npm package, but complains about trailing commas. i created a pull request that removes them

@melloware melloware reopened this Aug 2, 2023
@melloware
Copy link
Member

melloware commented Aug 2, 2023

You should now be able to do this in your package.json

"dependencies": {
        "primelocale": "primefaces/primelocale",

@Anders142536
Copy link
Contributor

Anders142536 commented Aug 3, 2023

The only issue we see in this solution now is that our company internal caches for depenency packages will not automatically work with github dependencies. I guess other people will have the same issue, so maybe uploading it to npm repo still makes sense.

@melloware
Copy link
Member

Agreed but according to the docs you can add a commit ID hash in your url and it will pull that version. Still not as good as in NPM but better than nothing

@melloware
Copy link
Member

@jerlam06
Copy link

Agreed but according to the docs you can add a commit ID hash in your url and it will pull that version. Still not as good as in NPM but better than nothing

@melloware What docs are you referring to please?

@melloware
Copy link
Member

melloware commented Nov 24, 2023

Sorry it's in the NPM docs for using just a git project like above. I had no idea you could do that.

PrimeTek is aware and is working on getting this released for real on NPMJS.

https://www.pluralsight.com/guides/install-npm-packages-from-gitgithub

@PotatoesMaster
Copy link

I guess this package is unofficial then.

@melloware
Copy link
Member

It is and I asked him to take it down. It's way out of date and should be released by PrimeTek.

@iwnow
Copy link

iwnow commented May 9, 2024

npm i github:primefaces/primelocale

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants