keymanapp / keymanapp/keyman

chore(common): build.sh deps could helpfully use `@keymanapp/<name>` package dependencies

Open
#12,421 0 comments 0 reactions 0 assignees View on GitHub
chore ci common/
Dominant language
Pascal
Stars
534
Forks
143
Avg merge
2d 10h
Merged PRs (30d)
113

Description

Currently, dependency definitions are always paths. It would be nice to alias those given we have that data in root package.json/dependencies (some need to be fixed up), e.g. @common/web/types == @keymanapp/common-types.

1. Allow build.sh to recognize `@node:keymanapp/` as a lookup for the mapping
2. Separate feature but helpful: add a linting step that greps for all import @keymanapp/* and reports on imports used but not referenced in one of package.json, tsconfig.json, build.sh (`/import(?:.+)from(?:.+)["']@keymanapp/(.+)["']/` in *.ts)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.