aboutsummaryrefslogtreecommitdiff
path: root/xtask/src/codegen/gen_parser_tests.rs
Commit message (Collapse)AuthorAgeFilesLines
* use strip_prefix() instead of starts_with and slicing (clippy::manual_strip)Matthias Krüger2021-03-211-6/+4
|
* Remove now dead codeAleksey Kladov2021-03-081-6/+6
|
* Simplify xtaskAleksey Kladov2021-03-011-1/+1
| | | | lib/bin/test separation isn't really needed.
* Deny unreachable-pubAleksey Kladov2020-11-021-5/+5
| | | | | | | | It's very useful when `pub` is equivalent to "this is crate's public API", let's enforce this! Ideally, we should enforce it for local `cargo test`, and only during CI, but that needs https://github.com/rust-lang/cargo/issues/5034.
* Use xshell::read_file instead of fs::read_to_stringIgor Aleksanov2020-10-191-1/+1
|
* Cleanup feature generationAleksey Kladov2020-08-181-4/+4
|
* Use anyhow::Result in xtask, add contextsPascal Hertleif2019-11-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This builds on #2231 but was actually done before that. You see, the cause for #2231 was that I got this error message: Error: Error { kind: Io(Os { code: 2, kind: NotFound, message: "No such file or directory" }) } Just switching to `anyhow::Result` got me stack traces (when setting `RUST_LIB_BACKTRACE=1`) that at least showed stack backtrace: 0: std::backtrace::Backtrace::create 1: std::backtrace::Backtrace::capture 2: anyhow::error::<impl core::convert::From<E> for anyhow::Error>::from 3: xtask::install_server 4: xtask::install 5: xtask::main 6: std::rt::lang_start::{{closure}} 7: std::panicking::try::do_call 8: __rust_maybe_catch_panic 9: std::rt::lang_start_internal 10: std::rt::lang_start 11: main With the added contexts (not at all exhaustive), the error became Error: install server Caused by: 0: build AutoCfg with target directory 1: No such file or directory (os error 2) Since anyhow is such a small thing (no new transitive dependencies!), and in general gives you `Result<T, Box<dyn Error>>` on steroids, I think this a nice small change. The only slightly annoying thing was to replace all the `Err(format!(…))?` calls (haven't even looked at whether we can make it support wrapping strings though), but the `bail!` macro is shorter anyway :)
* Some clippy fixeskjeremy2019-10-301-5/+3
|
* refactor comment extraction from tasksAleksey Kladov2019-10-241-38/+20
|
* simplifyAleksey Kladov2019-10-241-14/+13
|
* xtask: don't depend on itertoolsAleksey Kladov2019-10-231-17/+22
| | | | xtask should be fast to compiler, as it's a gateway to rust-analyzer
* xtask: move codegen to a moduleAleksey Kladov2019-10-231-0/+150