Move printing errors to nargo_cli

[kevaundray]

Problem

Currently errors are printed in noirc_driver to stdout, which is counter-intuitive for a crate that should be a library.

Happy Case

We can move the printing of errors to nargo_cli instead

Alternatives Considered

No response

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response

[ghost]

I will take care of it. Do I need to remove the printing from those places, right?

noir/crates/noirc_driver/src/lib.rs

Lines 194 to 200 in ae7ac21

	let main = match self.main_function() {
	Ok(m) => m,
	Err(e) => {
	println!("cannot compile a program with no main function");
	return Err(e);
	}
	};

noir/crates/noirc_driver/src/lib.rs

Lines 313 to 314 in ae7ac21

	let error = reporter::report(files, &err.into(), file, options.deny_warnings);
	reporter::finish_report(error as u32)?;

Try introducing a new error type.

[kevaundray]

@f01dab1e

I will take care of it. Do I need to remove the printing from those places, right?

Those print lines can stay for now, its mostly just about removing the error printing in noirc_driver and having the caller do the printing.

Try introducing a new error type.

Yeah I think we need to think about the Diagnostic and CustomDiagnostic types and whether the caller needs to have a FileManager.