-
Notifications
You must be signed in to change notification settings - Fork 4
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
NPM package versions #5
Comments
hi @bytestream, Thanks for the question and bring this to my attention as I wasn‘t aware. I‘ll share you context first before answering your question. Having Inter on Google Font has been a long wait dated back in March 2018 (so that we can use them with Kyle‘s typefaces package): google/fonts#1455 (comment) However I needed them for my projects so I decided to manually host them. We had a convo on Twitter which when the time comes (Inter available on Google Font), Kyle can just override the NPM package and make it work with his typefaces package: https://twitter.com/GatsbyJS/status/1093411095551692800 Also thanks for the fontsource recommendation. |
Thanks for the reply, context and your work on this package @ajmalafif Personally I don't think Kyle should be publishing v1's on this package name. I'll justify why:
Given the last point, for me, this package still has a use case. Perhaps @KyleAMathews can weight in. It would be preferable if those versions were removed and published under a different package name. NPM should be able to show you were anyone is actually pulling those v1's. Or at the very least, the README updated to note that there are two independent GitHub repositories publishing under separate version numbers simultaneously. |
Hi @bytestream, It took me awhile but I understand what‘s the issue is now and I am tempted to actually overwrite the NPM package to reflect this package. However I will probably mess things up as I am more of a designer than a programmer so I am worried of breaking stuffs unintentionally. Do you mind doing a PR (like should I have a release tag for this package?) and more than happy to push the changes to NPM. |
I'm not familiar with NPM. I don't think there's anything I can do from GitHub. To remove the versions I think you need to do one of the below:
See https://www.npmjs.com/policies/unpublish To update the README / linked repository, it looks like you need to publish a new version: |
Thanks for these insights! As you probably could tell I am not really familiar myself other than trying to figure out for my own use case. Give me a couple of days to make sure I don‘t break things and will update the package accordingly. |
No worries, take your time :) I just wanted to make you aware and find out why both versions exist |
Hi @bytestream, I‘ve finally found the courage and reverted the npm package back to as per Inter‘s version Feel free to correct if I‘ve done any mistake with the package, more than happy to fix it. Oh and Merry Christmas & happy holidays! |
Thanks very much! I'll let you know if I notice anything. Merry Christmas to you too! 🎅 |
@ajmalafif Please could you explain the versions at NPM?
https://www.npmjs.com/package/typeface-inter appears to be in use by both https://github.com/KyleAMathews/typefaces & this repository. This repository is using v3, the other is using v1. https://github.com/KyleAMathews/typefaces is also deprecated and recommends using https://github.com/fontsource/fontsource
The text was updated successfully, but these errors were encountered: