denoland / denoland/dnt

Better error message when test code has TLA for CJS distribution

Open
#111 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
1.3k
Forks
49
PR merge metrics
No merged PRs in 30d

Description

See #110. The error message should say if it is in test code.

Also, perhaps it would be useful to allow skipping running tests for CJS code? Just an idea that probably isn't worth the complexity.

```ts
{
test: "esm",
}
```

Contributor guide

Open the contributing guide

Research direction

Start by reading issue #110 and tracing how the `test: "esm"` configuration is handled when producing the CJS distribution. Confirm the current error path and identify whether it can distinguish test code; done means the error message explicitly says when the problem is in test code. The suggestion to skip CJS tests is an optional follow-up, not a defined requirement.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
build-system, testing
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.