Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP #17808

Closed
wants to merge 5 commits into from
Closed

WIP #17808

Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Prev

Parse Sizes attribute

  • Loading branch information
Rakhisharma committed Sep 25, 2017
commit 2f31ff0c638ada3acde9e90f25c61f809b2681c5
@@ -17,7 +17,7 @@ pub fn test_length_for_no_default_provided(len: f32) -> Length {

#[test]
fn no_default_provided() {
let a = vec![size];
let mut a = vec![];
let length = test_length_for_no_default_provided(100f32);
let size = Size { query: None, length: length };
a.push(size);
@@ -31,7 +31,7 @@ pub fn test_length_for_default_provided(len: f32) -> Length {

#[test]
fn default_provided() {
let a = vec![size];
let mut a = vec![];
let length = test_length_for_default_provided(2f32);
let size = Size { query: None, length: length };
a.push(size);

This comment has been minimized.

Copy link
@wafflespeanut

wafflespeanut Sep 24, 2017

Member

Nit: If it's just a single size, then let's have let a = vec![size]; (here and everywhere below)

@@ -56,7 +56,7 @@ pub fn test_length(len: f32) -> Length {

#[test]
fn one_value() {
let a = vec![size];
let mut a = vec![];
let media_query = test_media_query(200f32);
let length = test_length(545f32);
let size = Size { query: Some(media_query), length: length };
@@ -79,7 +79,7 @@ fn more_then_one_value() {

#[test]
fn no_extra_whitespace() {
let a = vec![size];
let mut a = vec![];
let media_query = test_media_query(200f32);
let length = test_length(545f32);
let size = Size { query: Some(media_query), length: length };
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.