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

Remove is_spotlight field from Trait #80914

Merged
merged 5 commits into from
Feb 24, 2021
Merged

Remove is_spotlight field from Trait #80914

merged 5 commits into from
Feb 24, 2021

Commits on Feb 23, 2021

  1. Remove is_spotlight field from Trait

    @GuillaumeGomez
    GuillaumeGomez committed Feb 23, 2021
    Configuration menu
    Copy the full SHA
    58a34a4 View commit details
    Browse the repository at this point in the history

  2. Use a qualified path to make it more clear where list_contains_name… 

    … function comes from
    @GuillaumeGomez
    GuillaumeGomez committed Feb 23, 2021
    Configuration menu
    Copy the full SHA
    fa51c0f View commit details
    Browse the repository at this point in the history

  3. Put clean::Trait extra information into a new struct to make it more … 

    …coherent
    @GuillaumeGomez
    GuillaumeGomez committed Feb 23, 2021
    Configuration menu
    Copy the full SHA
    b5c8eea View commit details
    Browse the repository at this point in the history

  4. * Fix some typo 

    * Improve documentation
* Add a test to ensure that spotlighted traits from dependencies are taken into account as expected
    @GuillaumeGomez
    GuillaumeGomez committed Feb 23, 2021
    Configuration menu
    Copy the full SHA
    3c59e64 View commit details
    Browse the repository at this point in the history

  5. Improve code readability

    @GuillaumeGomez
    GuillaumeGomez committed Feb 23, 2021
    Configuration menu
    Copy the full SHA
    33aaead View commit details
    Browse the repository at this point in the history