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

[rebber] Missing dependencies in package.json #384

Closed
thomasdondorf opened this issue Feb 24, 2020 · 9 comments · Fixed by #380
Closed

[rebber] Missing dependencies in package.json #384

thomasdondorf opened this issue Feb 24, 2020 · 9 comments · Fixed by #380
Labels
semver/bugfix This is a minor backward-compatible fix status/verified This has been checked by a maintainer type/meta This is not directly related to the code

Comments

@thomasdondorf
Copy link

Just tried to give rebber a try. Looks like it is missing two dependencies in the package.json file:

  • clone
  • trim-lines

After manually installing them I was able to get it to run. Used version was 5.0.0.

@StaloneLab StaloneLab added semver/bugfix This is a minor backward-compatible fix status/verified This has been checked by a maintainer type/meta This is not directly related to the code labels Feb 26, 2020
@StaloneLab StaloneLab linked a pull request Feb 26, 2020 that will close this issue
@StaloneLab
Copy link
Member

Hello there,

Thanks for reporting the issue, it will be solved in the next release of rebber coming soon. Feel free to ask if you have any questions about rebber or it's plugins.

@StaloneLab StaloneLab added this to the next release milestone Feb 26, 2020
@StaloneLab
Copy link
Member

Released today: rebber@5.1.0

@thomasdondorf
Copy link
Author

Nice, thanks for the fast feedback. Just to let you know: In the end, I didn't end up using rebber, mainly because of the missing TypeScript support. Also, I needed something to work directly on the AST.

Looks like a cool project, though :)

@artragis
Copy link
Member

Do you know how we could support typescript?

@StaloneLab
Copy link
Member

We need to provide typings I suppose.

@thomasdondorf
Copy link
Author

@artragis You could either put the type definition files into your project (like Unified did it) or provide them via DefinitelyTyped. Or you switch from JavaScript to TypeScript, but I guess that's not an option ;)

@LukeCarrier
Copy link

@artragis, @TitiAlone would you accept a PR adding TypeScript declarations for remark-abbr?

@StaloneLab
Copy link
Member

StaloneLab commented Oct 13, 2020

Please do; adding them to only some packages in the repo doesn't really make sense IMO, but I'm interested in seeing how typing declarations work. Do you think adding them to DefinitelyTyped would make sense, instead of adding them to this repository?

@LukeCarrier
Copy link

Awesome, I'll raise a PR for just one package for now and we can go from there. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/bugfix This is a minor backward-compatible fix status/verified This has been checked by a maintainer type/meta This is not directly related to the code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants