diff options
| author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-11-02 13:08:53 +0000 |
|---|---|---|
| committer | GitHub <[email protected]> | 2020-11-02 13:08:53 +0000 |
| commit | 731b38fa3c1694648e6c8e60f61820f9783343eb (patch) | |
| tree | b3f4e94e8eac9bb24296aacafacb100930bee59e /xtask/src | |
| parent | 6507877e7039d4517682a4fc232356662f509d81 (diff) | |
| parent | b6101184537b1165cfdd5fc473e04ad4c5b7bffa (diff) | |
Merge #6438
6438: Deny unreachable-pub r=matklad a=matklad
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.
Co-authored-by: Aleksey Kladov <[email protected]>
Diffstat (limited to 'xtask/src')
| -rw-r--r-- | xtask/src/codegen/gen_parser_tests.rs | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/xtask/src/codegen/gen_parser_tests.rs b/xtask/src/codegen/gen_parser_tests.rs index 19ae949d4..6e4abd10c 100644 --- a/xtask/src/codegen/gen_parser_tests.rs +++ b/xtask/src/codegen/gen_parser_tests.rs | |||
| @@ -45,15 +45,15 @@ pub fn generate_parser_tests(mode: Mode) -> Result<()> { | |||
| 45 | 45 | ||
| 46 | #[derive(Debug)] | 46 | #[derive(Debug)] |
| 47 | struct Test { | 47 | struct Test { |
| 48 | pub name: String, | 48 | name: String, |
| 49 | pub text: String, | 49 | text: String, |
| 50 | pub ok: bool, | 50 | ok: bool, |
| 51 | } | 51 | } |
| 52 | 52 | ||
| 53 | #[derive(Default, Debug)] | 53 | #[derive(Default, Debug)] |
| 54 | struct Tests { | 54 | struct Tests { |
| 55 | pub ok: HashMap<String, Test>, | 55 | ok: HashMap<String, Test>, |
| 56 | pub err: HashMap<String, Test>, | 56 | err: HashMap<String, Test>, |
| 57 | } | 57 | } |
| 58 | 58 | ||
| 59 | fn collect_tests(s: &str) -> Vec<Test> { | 59 | fn collect_tests(s: &str) -> Vec<Test> { |