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

feat(max_versions): set max versions based on string matching #369

Merged
merged 1 commit into from
Jun 6, 2019

Conversation

cnishina
Copy link
Contributor

  • Do a version match based on string startsWith of the maxVersion
    An example:

    Matching "0.1" might find versions 0.14.0, 0.151.0, 0.1.0. It will then
    get the max version which is 0.151. If a user wanted to get 0.1, then
    they could specify version "0.1."

  • Fix / update interchangable aliases:

    • selenium == standalone
    • chromedriver == chrome
    • geckodriver == gecko
    • iedriver = ie

Tagging #280, #358, #353. Additional work is still required to have a
config file that has this information instead of hard coding it.

@cnishina
Copy link
Contributor Author

cnishina commented Apr 24, 2019

DO NOT MERGE yet. Need to pass the test and write a unit test.

@cnishina cnishina force-pushed the max_versions branch 3 times, most recently from ee4a03d to 69c0450 Compare April 24, 2019 21:05
@cnishina cnishina requested a review from heathkit April 24, 2019 21:05
@cnishina cnishina force-pushed the max_versions branch 2 times, most recently from 16ca7f3 to bef4e3f Compare April 24, 2019 21:29
- Do a version match based on string startsWith of the maxVersion
  An example:

  Matching "0.1" might find versions 0.14.0, 0.151.0, 0.1.0. It will then
  get the max version which is 0.151. If a user wanted to get 0.1, then
  they could specify version "0.1."
- Added a unit test to the version_list to find max versions.

- Fix / update interchangable aliases:
  - selenium == standalone
  - chromedriver == chrome
  - geckodriver == gecko
  - iedriver = ie

Tagging angular#280, angular#358, angular#353. Additional work is still required to have a
config file that has this information instead of hard coding it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants