rokucommunity / rokucommunity/brighterscript
Warn when importing a d.bs file but using runtime stuff from them
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 208
- Forks
- 68
- Avg merge
- 8h 39m
- Merged PRs (30d)
- 39
Description
Sometimes you can accidentally import a .d.bs file instead of a .brs or .bs file. We should help developers detect this.
There are valid use cases for wanting to import a .d.bs file (maybe?). But if there's a .brs file next to it, perhaps we should warn you about using that one instead?
Or, we could just warn when using runtime files from an imported d.bs file because they won't exist at runtime.
Also, we probably shouldn't suggest importing .d.bs files at all?
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
The issue names no files, tests, or entry points. Start by locating import resolution and handling for .d.bs, .brs, and .bs files, then clarify which warning behavior is wanted and add coverage showing that the chosen warning appears only for runtime usage that will not exist at runtime.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- compilers
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 32/100