winglang/wing

PR linter

Open

#482 geöffnet am 7. Nov. 2022

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (214 Forks)github user discovery
good first issue👷‍♀️ contributor-experience

Repository-Metriken

Stars
 (5.385 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

PRs that change certain things should require new/changed example wing files to ensure coverage is available for the various tests that consume those files.

Feature Spec

Suggested projects to require test cases:

  • tree-sitter-wing
  • wingc (maybe only certain files, like the parser/typechecker/jsify)

Use Cases

Testing is often easy to forget, but for such foundational changes, we should nudge contributors to add bare minimum testing.

Implementation Notes

I think it would be interesting to look into generic tooling to support PR lints. e.g. https://danger.systems/js/

Also note we use mergify, which can also help

Component

  • Development Environment
  • Contributor Experience

Contributor Guide