Skip to content

add --compile-only flag for syntax/compile validation #6943

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Jawfish wants to merge 7 commits into from
Closed

add --compile-only flag for syntax/compile validation #6943

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
acc69e6
add --compile-only cli flag and runmode variant
Jawfish Feb 1, 2026
65f8f1e
implement compile-only mode handler
Jawfish Feb 1, 2026
ab1d681
add unit test for compile-only mode
Jawfish Feb 1, 2026
0fd1523
Merge branch 'main' into add-compile-only-flag
Jawfish Feb 1, 2026
b9d1330
fix --compile-only option termination
Jawfish Feb 1, 2026
2ad616c
exit non-zero when no files specified
Jawfish Feb 1, 2026
dde30cd
add docstring for compileonly variant
Jawfish Feb 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
implement compile-only mode handler
  • Loading branch information
@Jawfish
Jawfish committed Feb 1, 2026
commit 65f8f1e7d8861cc47de897e6c61216f4e1d0c7a9
34 changes: 33 additions & 1 deletion src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ fn run_rustpython(vm: &VirtualMachine, run_mode: RunMode) -> PyResult<()> {
RunMode::Module(_) => env::current_dir()
.ok()
.and_then(|p| p.to_str().map(|s| s.to_owned())),
RunMode::Script(_) | RunMode::InstallPip(_) => None, // handled by run_script
RunMode::Script(_) | RunMode::InstallPip(_) | RunMode::CompileOnly(_) => None, // handled by run_script
RunMode::Repl => Some(String::new()),
};

Expand Down Expand Up @@ -287,6 +287,38 @@ fn run_rustpython(vm: &VirtualMachine, run_mode: RunMode) -> PyResult<()> {
run_file(vm, scope.clone(), &script_path)
}
RunMode::Repl => Ok(()),
RunMode::CompileOnly(files) => {
debug!("Compiling {} file(s)", files.len());
if files.is_empty() {
eprintln!("No files specified for --compile-only");
return Ok(());
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
let mut success = true;
for file in files {
match std::fs::read_to_string(&file) {
Ok(source) => {
if let Err(err) =
vm.compile(&source, vm::compiler::Mode::Exec, file.clone())
{
eprintln!("Error compiling {file}: {err}");
success = false;
}
}
Err(err) => {
eprintln!("Error reading {file}: {err}");
success = false;
}
}
}
if !success {
let exit_code = vm.ctx.new_int(1);
return Err(vm.new_exception(
vm.ctx.exceptions.system_exit.to_owned(),
vec![exit_code.into()],
));
}
Ok(())
}
};
let result = if is_repl || vm.state.config.settings.inspect {
shell::run_shell(vm, scope)
Expand Down