Skip to content
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

687 sudo #699

Merged
merged 32 commits into from
Sep 6, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
5bdc1b4
Merge pull request #17 from google/master
PopGoesTheWza May 13, 2019
ed24bb7
Merge pull request #18 from google/master
PopGoesTheWza May 14, 2019
807729c
2.1.0
grant Mar 19, 2019
83b9e78
Low impact changes (#619)
PopGoesTheWza May 17, 2019
a89ab84
Relative rootdir (#620)
PopGoesTheWza May 17, 2019
19a2fd6
Cleanup before next pr (#621)
PopGoesTheWza May 20, 2019
2abec30
Refactor `inquirer` package (#622)
PopGoesTheWza May 21, 2019
4829c18
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza May 22, 2019
4b77408
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza May 23, 2019
cd4b7cc
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza May 24, 2019
5279445
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza May 29, 2019
4b0fe1c
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jun 3, 2019
6aca82a
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jun 14, 2019
ec9f3e8
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jun 14, 2019
81d3792
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jun 26, 2019
9a911f5
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jul 1, 2019
4cd43a9
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jul 3, 2019
8018780
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jul 3, 2019
f3706ea
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jul 4, 2019
f468920
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jul 15, 2019
9cac474
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jul 22, 2019
e852032
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Jul 26, 2019
df340af
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Aug 1, 2019
dc88dd2
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Aug 8, 2019
f498dba
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Aug 16, 2019
a9931b8
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Aug 21, 2019
c14e765
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Aug 21, 2019
4d3ae22
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Aug 26, 2019
e544b03
Merge branch 'master' of https://github.com/google/clasp
PopGoesTheWza Aug 30, 2019
8a9d486
unnecessary `sudo`
PopGoesTheWza Sep 6, 2019
4da314f
package update
PopGoesTheWza Sep 6, 2019
d409dd3
node version req & scripts update
PopGoesTheWza Sep 6, 2019
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ clear
First download `clasp`:

```sh
sudo npm i @google/clasp -g
npm install -g @google/clasp
# Alternatively: sudo npm i -g grpc @google/clasp --unsafe-perm
```

Expand Down Expand Up @@ -521,12 +521,12 @@ Specifies the files that should be pushed first, useful for scripts that rely on

### Node Version

The library requires **Node version >= 6.3.0**. Use this script to check your version and **upgrade Node if necessary**:
The library requires **Node version >= 8.2.1**. Use this script to check your version and **upgrade Node if necessary**:

```sh
node -v # Check Node version
sudo npm install n -g
sudo n latest
npm install -g npm # Update npm and npx
npx n latest # use the n package to update node
```

### Using a Proxy
Expand Down
14 changes: 7 additions & 7 deletions docs/develop.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,14 @@ You can develop and test `clasp` on your computer by following these steps.
### Setup

- Install `tsc`: `npm install -g typescript`
- Remove your local version of `clasp`: `sudo npm uninstall -g @google/clasp`
- Remove your local version of `clasp`: `npm uninstall -g @google/clasp`
- This will prevent errors when updating `node_modules`.
- Install dependencies: `npm i`
- Install dependencies: `npm install`

### After Making a Change

```sh
sudo npm run build;
npm run build;
clasp <command>
```

Expand All @@ -23,9 +23,9 @@ clasp <command>
If you're seeing build errors, try deleting `node_modules` and building `clasp` from scratch:

```sh
sudo rm package-lock.json
sudo rm -rf node_modules/
sudo npm run build-fresh
rm package-lock.json
rm -rf node_modules/
npm run build-fresh
```

This is what @grant's terminal looks like:
Expand Down Expand Up @@ -55,7 +55,7 @@ After seeing that message, you're ready to test out `clasp`!
`clasp` has some unit tests that help detect errors. Build and run tests with these commands:

```sh
sudo npm run build;
npm run build;
npm run test
```

Expand Down
26 changes: 13 additions & 13 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

14 changes: 9 additions & 5 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,18 @@
"description": "Develop Apps Script Projects locally",
"main": "./src/index.js",
"scripts": {
"build": "tsc --project tsconfig.json && npm i -g --loglevel=error",
"build": "npm run build-local && npm i -g --loglevel=error",
"build-fresh": "npm cache clean --force && npm i && npm run build",
"build-local": "tsc --project tsconfig.json",
"watch": "tsc --project tsconfig.json --watch",
"publish": "npm publish --access public",
"lint": "tslint --project tslint.json && echo 'No lint errors. All good!'",
"test": "nyc ./node_modules/.bin/mocha -r ts-node/register --cache false --timeout 100000 'tests/**/*.ts'",
"test": "nyc mocha -r ts-node/register --cache false --timeout 100000 'tests/**/*.ts'",
"coverage": "nyc --cache false report --reporter=text-lcov | coveralls",
"prettier": "./node_modules/prettier/bin-prettier.js --parser typescript --single-quote --bracket-spacing --print-width 110 --trailing-comma all src/**/*.ts --write"
"prettier": "prettier --parser typescript --single-quote --bracket-spacing --print-width 110 --trailing-comma all src/**/*.ts --write"
},
"engines": {
"node": ">=8.2.1"
},
"bin": {
"clasp": "./src/index.js"
Expand Down Expand Up @@ -65,7 +69,7 @@
"fs-extra": "^8.1.0",
"fuzzy": "^0.1.3",
"gaxios": "^2.0.1",
"google-auth-library": "^5.2.0",
"google-auth-library": "^5.2.1",
"googleapis": "^42.0.0",
"inquirer": "^7.0.0",
"inquirer-autocomplete-prompt": "1.0.1",
Expand All @@ -83,7 +87,7 @@
"watch": "^1.0.2"
},
"devDependencies": {
"@types/chai": "^4.2.0",
"@types/chai": "^4.2.1",
"@types/cli-spinner": "^0.2.0",
"@types/fs-extra": "^8.0.0",
"@types/inquirer": "^6.5.0",
Expand Down
2 changes: 1 addition & 1 deletion tests/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
## How to Run Tests

1. Log in: `clasp login`
1. Rebuild: `sudo npm run build`
1. Rebuild: `npm run build`
1. Set environmental variables:
- `export TRAVIS_PULL_REQUEST=false`
- `export SCRIPT_ID=1EwE84eZCSBPcaAiJzCnDjmxMVnLQrDyhSKq1oZY6q-3x4BIDHgQefCnL`
Expand Down