8000 translate "latest" ami_version to "2.4.2" · Issue #1269 · Yelp/mrjob · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
translate "latest" ami_version to "2.4.2" #1269
New issue
New issue
Closed
Closed
translate "latest" ami_version to "2.4.2"#1269
Labels
Cleanup
Milestone
 
v0.5.0
@coyotemarin

Description

@coyotemarin
coyotemarin
opened on Mar 18, 2016
< 639B button data-component="IconButton" type="button" aria-haspopup="true" aria-expanded="false" tabindex="0" class="prc-Button-ButtonBase-c50BI prc-Button-IconButton-szpyj" data-loading="false" data-no-visuals="true" data-size="small" data-variant="invisible" aria-describedby=":R5ldklmb:-loading-announcement" aria-labelledby=":R3tldklmb:" id=":R5ldklmb:">

The AMI version "latest" is very, very deprecated, and is unlikely to ever mean anything but 2.4.2.

Currently, we have to explicitly check for "latest" in several points in our code, to protect the version_gte() function on Python 3 (doesn't like string-to-number comparisons).

Instead, let's just translate latest to 2.4.2 up-front.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Cleanup

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions
      0