Skip to content
This repository was archived by the owner on Mar 11, 2025. It is now read-only.

Fix expected error message for missing file #512

Merged
merged 1 commit into from
Jul 2, 2021
Merged

Fix expected error message for missing file #512

merged 1 commit into from
Jul 2, 2021

Conversation

Kijewski
Copy link
Member

@Kijewski Kijewski commented Jul 2, 2021

rust-lang/rust#82069 made error message that stem macro invocations more
verbose. Since Rust 1.54 (currently in beta) the message includes the
name of the offending macro.

This PR uses version_check to select the appropriate expected error
message.

rust-lang/rust#82069 made error message that stem macro invocations more
verbose. Since Rust 1.54 (currently in beta) the message includes the
name of the offending macro.

This PR uses version_check to select the appropriate expected error
message.
@djc djc merged commit c31fe5f into askama-rs:main Jul 2, 2021
@djc
Copy link
Collaborator

djc commented Jul 2, 2021

Thanks! In future work, for if/else expressions where the contained block is a single expression that fits on one line, please consider matching the boolean instead.

@Kijewski Kijewski deleted the pr-pull-82069 branch July 2, 2021 14:17
@Kijewski
Copy link
Member Author

Kijewski commented Jul 2, 2021

Will do.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants