Skip to content

feat: ignore unresolved import patterns#13

Merged
longlho merged 1 commit into
mainfrom
codex/ignore-unresolved-imports
May 18, 2026
Merged

feat: ignore unresolved import patterns#13
longlho merged 1 commit into
mainfrom
codex/ignore-unresolved-imports

Conversation

@longlho
Copy link
Copy Markdown
Contributor

@longlho longlho commented May 18, 2026

Summary

  • Fold unresolved import ignore patterns into the unresolvedImports config object.
  • Remove the separate ignoreUnresolved top-level option.
  • Keep mode and ignored unresolved specifier patterns in one config surface.

Example

{
  "unresolvedImports": {
    "mode": "report",
    "ignore": ["#virtual_generated/**"]
  }
}

Test plan

  • cargo fmt --check
  • git diff --check
  • cargo test --workspace
  • bazel test --test_output=errors //tests/fixtures/runfiles-fixture:codescythe_runfiles_test //tests/fixtures/runfiles-false-positive:codescythe_false_positive_test

@longlho longlho force-pushed the codex/unresolved-import-modes branch from 0b04bf3 to 5497773 Compare May 18, 2026 14:40
@longlho longlho force-pushed the codex/ignore-unresolved-imports branch from 3bbcae2 to e6b1908 Compare May 18, 2026 14:40
@longlho longlho force-pushed the codex/unresolved-import-modes branch from 5497773 to e9b4a56 Compare May 18, 2026 14:47
@longlho longlho force-pushed the codex/ignore-unresolved-imports branch from e6b1908 to dd0dce7 Compare May 18, 2026 14:47
Base automatically changed from codex/unresolved-import-modes to main May 18, 2026 14:51
@longlho longlho force-pushed the codex/ignore-unresolved-imports branch from dd0dce7 to 2f1f866 Compare May 18, 2026 14:54
@longlho longlho force-pushed the codex/ignore-unresolved-imports branch from 2f1f866 to 693727b Compare May 18, 2026 15:09
@longlho longlho merged commit c9569b0 into main May 18, 2026
5 checks passed
@longlho longlho deleted the codex/ignore-unresolved-imports branch May 18, 2026 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant