Skip to content

fix: Missing semicolon in Custom Media Queries#260

Merged
devongovett merged 3 commits intoparcel-bundler:masterfrom
Marabyte:fix/custom-media-queries
Aug 25, 2022
Merged

fix: Missing semicolon in Custom Media Queries#260
devongovett merged 3 commits intoparcel-bundler:masterfrom
Marabyte:fix/custom-media-queries

Conversation

@Marabyte
Copy link
Contributor

@Marabyte Marabyte commented Aug 24, 2022

fix #258

Adds a semicolon at the end of the @custom-media declaration.

@devongovett thank you for pointing me to the right file!

@Marabyte Marabyte changed the title Fix/custom-media-queries fix: Missing semicolon in Custom Media Queries Aug 24, 2022
Copy link
Member

@devongovett devongovett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

cmd.assert().success().stdout(predicate::str::contains(indoc! {r#"
@custom-media --foo print;
"#}));

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For next time, you could also have added this as a unit test - they are slightly easier to write and live in src/lib.rs. This is also fine though!

@devongovett devongovett merged commit 6efa74d into parcel-bundler:master Aug 25, 2022
@Marabyte Marabyte deleted the fix/custom-media-queries branch August 28, 2022 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Preserve unused @custom-media

2 participants