Skip to content

chore: drop *.rust from fixture .gitignore - #4

Merged
Kinflou merged 1 commit into
mainfrom
fix/gitignore-rust
Aug 28, 2026
Merged

chore: drop *.rust from fixture .gitignore#4
Kinflou merged 1 commit into
mainfrom
fix/gitignore-rust

Conversation

@Kinflou

@Kinflou Kinflou commented Aug 28, 2026

Copy link
Copy Markdown
Member

.rust is never a valid generated extension — the generator emits .rs (see #3 / ComlineProject/core#22). Ignoring *.rust only masked a stale-binary situation. Keeps *.rs, which is the real generated output for this .ids-only fixture.

.rust is never a valid generated extension (the generator emits .rs);
ignoring it only masked the stale-binary bug. Keep *.rs, which is the
real generated output for this .ids-only fixture.
@Kinflou
Kinflou merged commit aae7eae into main Aug 28, 2026
2 checks passed
@Kinflou
Kinflou deleted the fix/gitignore-rust branch August 28, 2026 05:45
Kinflou pushed a commit that referenced this pull request Aug 31, 2026
generate.rs now speaks the GeneratedFile contract (generation#5):

- t.mode string -> Mode (code / lib; dylib rejected).
- Per version, build a GenRequest with every schema, the mode, and
  PackageMeta { name = congregation name, version = package version or
  0.0.0 for an unbuilt tree }, call the generator once, write the files
  it returns.
- code mode: one file per schema, still placed by the layout template.
- lib mode: the returned crate goes under <out>/<language>/ verbatim
  (Cargo.toml + src/lib.rs + src/<namespace>.rs). The layout template is
  not consulted inside the crate.
- lib + multiple package versions -> error for now.

Cargo.toml: bump comline-codelib-gen to db4f166 (lib mode + autobins fix
+ the TypeScript generator from #4).

Tests: lib_mode_emits_a_crate, dylib_mode_is_rejected. 38 pass.

Follow-ups: a namespace literally named main or lib still triggers Rust's
special_module_name warning (nest schema modules under src/schemas/);
clean.rs could ask the generator for its paths rather than lean on the
"dedicated out dir" branch.
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