Skip to content

test: add basic Lua, Erlang, and Fortran samples - #1434

Merged
reyammer merged 1 commit into
google:mainfrom
vivekjm:test/add-basic-language-samples
Sep 7, 2026
Merged

reyammer merged 1 commit into
google:mainfrom
vivekjm:test/add-basic-language-samples

Conversation

@vivekjm

@vivekjm vivekjm commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add basic Lua, Erlang, and Fortran samples for content-type regression coverage
  • keep each example small while including characteristic syntax for its language

Contributes to #662.

Sample provenance

I wrote all three text files from scratch for this pull request. They use fictional weather readings and station names, and do not include any third-party or generated source material.

Testing

  • magika --json tests_data/basic/lua/weather.lua tests_data/basic/erlang/weather_station.erl tests_data/basic/fortran/temperature_report.f90 — all three returned the expected labels with the standard_v3_3 model
  • python/scripts/run_quick_test_magika_cli.py — passed against the full basic corpus
  • python/scripts/run_quick_test_magika_module.py — passed against the full basic corpus
  • go test ./... (from go/) — passed
  • git diff --check — passed

AI assistance

I used OpenAI Codex to help identify unclaimed content types, draft the sample files, and run the repository checks. I reviewed the final files, verified their behavior locally, and will respond to review feedback myself.

@vivekjm
vivekjm requested a review from reyammer as a code owner August 13, 2026 07:34
@reyammer
reyammer force-pushed the test/add-basic-language-samples branch from 95999b6 to b5458ce Compare September 7, 2026 12:53
@reyammer
reyammer merged commit 2b04d89 into google:main Sep 7, 2026
110 checks passed
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.

2 participants