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
Adds more instructions for develop in Readme #120
Conversation
Also updates with clasp login --ownkey Signed-off-by: campionfellin <campionfellin@gmail.com>
2 similar comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ownkey
sentence looks good.Run Tests
description could use improvement.- Just state how to run the tests.
README.md
Outdated
@@ -213,6 +215,9 @@ clasp <command> | |||
|
|||
#### Run Tests (experimental) | |||
|
|||
Check out `/tests/test.ts` and change `describe.skip(...)` to `describe(...)` for relevant tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a better way to run individual tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't say Check out /tests/test.ts\
. It's wordy. Accommodate for most people reading this.
I'm guessing if someone searches for this section
90% of people want to run tests.
10% of people want to add a test.
So just state how to run all tests. 1 sentence.
If you find a way to run a single test, you can state that too in 1 sentence/command.
README.md
Outdated
@@ -213,6 +215,9 @@ clasp <command> | |||
|
|||
#### Run Tests (experimental) | |||
|
|||
Check out `/tests/test.ts` and change `describe.skip(...)` to `describe(...)` for relevant tests. | |||
If you created a new command, please make a test for it. Here's how to run the tests: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete: If you created a new command, please make a test for it.
We can add this when tests are common and include it somewhere else.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ownkey
sentence looks good.Run Tests
description could use improvement.- Just state how to run the tests.
Signed-off-by: campionfellin <campionfellin@gmail.com>
Much better, thanks! |
Also updates with clasp login --ownkey
Signed-off-by: campionfellin campionfellin@gmail.com