Skip to content

Update ASIM warning baseline - #167

Closed
zedoraps wants to merge 1 commit into
mainfrom
fix/asim-warning-baseline
Closed

Update ASIM warning baseline#167
zedoraps wants to merge 1 commit into
mainfrom
fix/asim-warning-baseline

Conversation

@zedoraps

@zedoraps zedoraps commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

🔍 Problem

  • The package tests on main fail because released Tenzir now emits diagnostic source locations for the existing unsupported OCSF-to-ASIM mapping warning.
  • The warning itself is not new; only the rendered diagnostic context changed.

🛠️ Solution

  • Regenerate the ASIM OCSF warning baseline using the same root-project invocation shape as CI.

💬 Review

  • Verify that the expected diagnostic paths are repository-relative.
  • Local check: uvx tenzir-test .

Regenerate the Microsoft ASIM OCSF warning fixture with tenzir-test so it matches the diagnostic locations emitted by the released Tenzir CLI.

Assisted-by: GPT-5 (Codex)
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@zedoraps

zedoraps commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

Closing as duplicate: the same ASIM warning baseline update was already merged via #166.

@zedoraps zedoraps closed this Jul 7, 2026
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