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
Move to yarn #685
Move to yarn #685
Conversation
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.
Just a few comments & small requests
docs/getting_started.md
Outdated
$ npm install | ||
$ npm run build | ||
$ yarn install | ||
$ yarn build |
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.
Only yarn install
(or just yarn
) is needed, as yarn build
gets run automatically by virtue of being the "prepare"
script in package.json
.
docs/getting_started.md
Outdated
@@ -14,8 +14,8 @@ work for older releases and other homeservers, but configuration may vary. | |||
```sh | |||
$ git clone https://github.com/matrix-org/matrix-appservice-slack.git | |||
$ cd Matrix-appservice-slack |
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.
Aside: the M should be lower case
docs/getting_started.md
Outdated
@@ -73,7 +73,7 @@ ever stuck, you can post a question in the | |||
port it will send messages through (if this bridge runs on the same | |||
machine you can use `localhost` as the `$HOST` name): | |||
|
|||
`$ npm start -- -r -c config/config.yaml -u "http://$HOST:$MATRIX_PORT"` | |||
`$ yarn start -- -r -c config/config.yaml -u "http://$HOST:$MATRIX_PORT"` |
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.
yarn [run] <script>
doesn't require --
for option separation, and even dumps a warning about it:
warning From Yarn 1.0 onwards, scripts don't require "--" for options to be forwarded. In a future version, any explicit "--" will be forwarded as-is to the scripts.
package.json
Outdated
"start": "node ./lib/app.js", | ||
"build": "tsc --build", | ||
"test": "npm run test:unit && npm run test:integration", | ||
"test": "yarn run test:unit && yarn run test:integration", |
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.
yarn run <script>
can be just yarn <script>
.
b521494
to
5d4ed48
Compare
* Update appservice-bridge, ping yargs * Move to yarn * changelog * Ensure we use yarn in all the places * Drop package-lock from dockerfile * Use 4.0.2 * Don't try to build in prod * Requested changes
To be inline with the rest of our projects.
CI will fail due to buildkite not being updated. See #688 for working CI.