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.

Sign up for GitHub

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

Jump to bottom

Split class names on whitespace when using tuple or Vec #1084

Merged
merged 3 commits into from Apr 14, 2020
Merged

Split class names on whitespace when using tuple or Vec #1084

merged 3 commits into from Apr 14, 2020

Commits on Apr 14, 2020

  1. Add tests for current behavior of Classes

    @bryanjswift
    bryanjswift committed Apr 14, 2020
    Configuration menu
    Copy the full SHA
    c8e1fba View commit details
    Browse the repository at this point in the history

  2. Change Classes::push to split incoming class on whitespace

    @bryanjswift
    bryanjswift committed Apr 14, 2020
    Configuration menu
    Copy the full SHA
    14b00b6 View commit details
    Browse the repository at this point in the history

  3. Split each item in Vec on whitespace in From<Vec<T>> for Classes 

    When creating a `VTag` with `class` assigned from a `Vec` the items in
`Vec` were not split on whitespace and so were not enforced as unique in
the same way a `VTag` with `class` assigned from `tuple`.
    @bryanjswift
    bryanjswift committed Apr 14, 2020
    Configuration menu
    Copy the full SHA
    ccca233 View commit details
    Browse the repository at this point in the history