Merge pull request #12 from laughingman7743/fix_invalid_type_for_regi…

…stry_ids

Fix invalid type for registry ids (fix #11)

README.rst

@@ -51,7 +51,7 @@ profile name, region name, registry ID and tag can be set.
 +--------------+------------+-------------------------------------------------------------------------------------+
 | region_name  | str        | The region to use. Overrides config/env settings.                                   |
 +--------------+------------+-------------------------------------------------------------------------------------+
-| registry_id  | str        | AWS account ID that correspond to a Amazon ECR registry that you want to log in to. |
+| registry_id  | int        | AWS account ID that correspond to a Amazon ECR registry that you want to log in to. |
 +--------------+------------+-------------------------------------------------------------------------------------+
 | tags         | seq of str | Name and optionally a tag in the ‘name:tag’ format.                                 |
 +--------------+------------+-------------------------------------------------------------------------------------+
@@ -77,11 +77,11 @@ Usage
     Usage: ecr [OPTIONS] COMMAND [ARGS]...
 
     Options:
-      --profile TEXT        Use a specific profile from your credential file.
-      --region TEXT         The region to use. Overrides config/env settings.
-      --registry-id TEXT    AWS account ID that correspond to a Amazon ECR registry that you want to log in to.
-      --debug / --no-debug  Turn on debug logging.
-      -h, --help            Show this message and exit.
+      --profile TEXT         Use a specific profile from your credential file.
+      --region TEXT          The region to use. Overrides config/env settings.
+      --registry-id INTEGER  AWS account ID that correspond to a Amazon ECR registry that you want to log in to.
+      --debug / --no-debug   Turn on debug logging.
+      -h, --help             Show this message and exit.
 
     Commands:
       build  Build an image from a Dockerfile.

ecr_cli/cli.py

@@ -17,7 +17,7 @@
               help=msg.HELP_OPTION_PROFILE)
 @click.option('--region', type=str, required=False,
               help=msg.HELP_OPTION_REGION)
-@click.option('--registry-id', type=str, required=False,
+@click.option('--registry-id', type=int, required=False,
               help=msg.HELP_OPTION_REGISTRY_ID)
 @click.option('--debug/--no-debug', required=False, default=False,
               help=msg.HELP_OPTION_DEBUG)

ecr_cli/model.py

@@ -18,7 +18,7 @@ def from_dict(value):
             tags,
             value.get('profile_name', None),
             value.get('region_name', None),
-            value.get('registry_id', None),)
+            int(value.get('registry_id', None)),)
 
     def _key(self):
         return (self.tags,