exercism / exercism/cli

snap package prints erroneous token config instructions

オープン
#905 コメント 5 件 リアクション 11 件 担当者 0 名 GitHub で見る
主要言語
Go
スター
1.4k
フォーク
363
平均マージ
6日 13時間
マージ済み PR(30日)
2

説明

I haven't used exercism for a while, but upon deciding to revisit after a hiatus, I was pleased to find there is now a snap package available. Following the instructions at

https://exercism.io/cli-walkthrough

```
> sudo snap install exercism
```

and then, admittedly, not completely following all the instructions, I jumped to trying to install an exercise, and got the following error message

```
> exercism download --exercise=hello-world --track=typescript
Error:

Welcome to Exercism!

To get started, you need to configure the tool with your API token.
Find your token at

https://exercism.io/my/settings

Then run the configure command:

/snap/exercism/5/exercism configure --token=YOUR_TOKEN
```

In copying the instructions and replacing with my token

```
/snap/exercism/5/exercism configure --token=
```

and getting a positive confirmation message that it had worked correctly, I tried re-downloading the exercise

```
> exercism download --exercise=hello-world --track=typescript
```

only to receive the same error message. After a minor amount of debugging to determine the path of the actual `exercism` binary

```
> which exercism
/snap/bin/exercism
```

and using the updated path when running configure

```
/snap/bin/exercism configure --token=
```

it worked. Ultimately, I stumbled upon the correct instruction on the third page of

https://exercism.io/cli-walkthrough

```
exercism configure --token=
```

But it seems like the command

```
> exercism download --exercise= --track=
```

should output this instead of the ` /snap/exercism/5/exercism` path.

I realize I probably should have RTFM and I don't know enough about snaps to know how hard it is to fix, but seems like a reasonable fix given it's a beginner tool and the much appreciated effort for creating the snap.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。