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

Fix artifact selection bug #53

Merged
merged 5 commits into from
May 25, 2022
Merged

Fix artifact selection bug #53

merged 5 commits into from
May 25, 2022

Commits on May 25, 2022

  1. Log decisions around asset name choice

    @ZoteTheMighty
    ZoteTheMighty committed May 25, 2022
    Configuration menu
    Copy the full SHA
    476cad3 View commit details
    Browse the repository at this point in the history

  2. Factor out code that needs separate testing

    @ZoteTheMighty
    ZoteTheMighty committed May 25, 2022
    Configuration menu
    Copy the full SHA
    56a2b7e View commit details
    Browse the repository at this point in the history

  3. swap loops so that we find best match for most specific keyword befor… 

    …e we find the first asset that matches ANY keyword
    @ZoteTheMighty
    ZoteTheMighty committed May 25, 2022
    Configuration menu
    Copy the full SHA
    c5d8059 View commit details
    Browse the repository at this point in the history

  4. Make test a bit more thorough

    @ZoteTheMighty
    ZoteTheMighty committed May 25, 2022
    Configuration menu
    Copy the full SHA
    075afea View commit details
    Browse the repository at this point in the history

  5. cargo fmt

    @ZoteTheMighty
    ZoteTheMighty committed May 25, 2022
    Configuration menu
    Copy the full SHA
    3cc8ac5 View commit details
    Browse the repository at this point in the history