dart-lang / dart-lang/source_gen
Make LibraryReader.pathToUrl take a non-null argument
Open
next-breaking-release
- Dominant language
- Dart
- Stars
- 494
- Forks
- 115
- Avg merge
- 12h 13m
- Merged PRs (30d)
- 3
Description
It is statically breaking to move the runtime null check to a static null check, but it's worth doing because a static check is preferred over a runtime check.
Contributor guide
Research direction
Locate LibraryReader.pathToUrl in the source_gen code and inspect its callers and current nullability checks to understand the API impact. Done means the argument is statically non-null and the affected code still passes static analysis; the issue does not mention a specific file or test.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- dart
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100