Merge pull request #17 from hiro-o918/feature/readme

Update README and format code style

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "awsctx"
-version = "0.2.3"
+version = "0.2.4"
 authors = ["Hironoiri Yamamoto <[email protected]>"]
 edition = "2018"
 description = "Context Manager for AWS Profiles"

README.md

@@ -47,19 +47,11 @@ aws_secret_access_key = YYYYYYYYYYY
 aws_session_token = YYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYY
 ```
 
-### List Profiles
-`list-contexts` shows all the named profiles as the below.
-```console
-$ ctxm list-contexts
-  foo
-  bar
-```
-
-### Use Context
+### Update the default profile by CLI
 `use-context` update `~/.aws/credentials` to contain `default` profile that the values are same as specified in the option.
 
 ```console
-$ ctxm use-context -p foo
+$ awsctx use-context -p foo
 ```
 e.g. the above commands updates credentials as the below.
 ```

src/aws.rs

@@ -28,9 +28,9 @@ impl ctx::CTX for AWS {
         let creds = Credentials::load_credentials(&self.credentials_path)?;
         let profiles = creds.list_profiles();
         Ok(profiles
-            .iter()
+            .into_iter()
             .map(|p| ctx::Context {
-                name: p.name.clone(),
+                name: p.name,
                 active: p.default,
             })
             .collect())
@@ -47,14 +47,18 @@ impl ctx::CTX for AWS {
     }
 
     fn use_context_interactive(&self) -> Result<ctx::Context, ctx::CTXError> {
-        let mut contexts = self.list_contexts().unwrap();
+        let mut contexts = self.list_contexts()?;
         // skim shows reverse order
         contexts.reverse();
         let options = SkimOptionsBuilder::default()
             .height(Some("30%"))
             .multi(false)
             .build()
-            .unwrap();
+            .or_else(|err| {
+                Err(ctx::CTXError::UnexpectedError {
+                    source: anyhow!(err),
+                })
+            })?;
 
         let (tx_item, rx_item): (SkimItemSender, SkimItemReceiver) = unbounded();
         for context in contexts {

src/creds.rs

@@ -14,6 +14,7 @@ const DEFAULT_PROFILE_KEY: &str = "[default]";
 pub struct Profile {
     pub name: String,
     pub default: bool,
+    #[allow(dead_code)]
     items: Vec<String>,
 }