Skip to content

Commit d87c83a

Browse files
charan678sr010shrutiburman
authored
fix!: Replace oclif-v1 dependencies with oclif-v2 version (twilio#212)
* fix!: update description (twilio#207) * Update twilio-client.test.js * Update base-command.js * fix: Fixing ocktokit api calls (twilio#211) * Update update-release.js * fix: replace oclif v1 dependecies to oclif v2 dependencies * BREAKING CHANGE: unlocking oclif v2 Co-authored-by: sr010 <[email protected]> Co-authored-by: sburman <[email protected]> Co-authored-by: shrutiburman <[email protected]>
1 parent ba6a147 commit d87c83a

File tree

4 files changed

+8
-10
lines changed

4 files changed

+8
-10
lines changed

PULL_REQUEST_TEMPLATE.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ We appreciate the effort for this pull request but before that please make sure
44
Please format the PR title appropriately based on the type of change:
55
<type>[!]: <description>
66
Where <type> is one of: docs, chore, feat, fix, test.
7-
Add a '!' after the type for breaking changes (e.g. feat!: new breaking feature).
7+
Add a '!' after the type for breaking changes (e.g. feat!: new breaking feature) & add text "BREAKING CHANGE:" in commit msg.
88
99
**All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.**
1010

package.json

+2-4
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,6 @@
2929
"dependencies": {
3030
"@actions/core": "^1.0.0",
3131
"@actions/github": "^5.0.3",
32-
"@oclif/command": "^1.8.15",
33-
"@oclif/config": "^1.18.2",
34-
"@oclif/errors": "^1.3.5",
3532
"@oclif/plugin-help": "^3.2.2",
3633
"@oclif/plugin-plugins": "^1.8.3",
3734
"@octokit/rest": "^19.0.3",
@@ -44,7 +41,8 @@
4441
"qs": "^6.9.4",
4542
"semver": "^7.3.2",
4643
"tsv": "^0.2.0",
47-
"twilio": "^3.80.0"
44+
"twilio": "^3.80.0",
45+
"@oclif/core": "^1.13.1"
4846
},
4947
"devDependencies": {
5048
"@oclif/test": "^1.2.6",

src/base-commands/base-command.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
const { Command, flags: oclifFlags } = require('@oclif/command');
2-
const { CLIError } = require('@oclif/errors');
1+
const { Command, Flags: oclifFlags } = require('@oclif/core');
2+
const { Errors } = require('@oclif/core');
33

44
const pkg = require('../../package.json');
55
const MessageTemplates = require('../services/messaging/templates');
@@ -30,7 +30,7 @@ class BaseCommand extends Command {
3030
}
3131

3232
async run() {
33-
const { args, flags } = this.parse(this.constructor);
33+
const { args, flags } = await this.parse(this.constructor);
3434
this.args = args;
3535
this.flags = flags;
3636
await this.loadConfig();
@@ -59,7 +59,7 @@ class BaseCommand extends Command {
5959
}
6060

6161
async catch(error) {
62-
if (!this.logger || instanceOf(error, CLIError)) {
62+
if (!this.logger || instanceOf(error, Errors.CLIError)) {
6363
return super.catch(error);
6464
}
6565

src/base-commands/twilio-client-command.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
const { flags } = require('@oclif/command');
1+
const { Flags: flags } = require('@oclif/core');
22

33
const BaseCommand = require('./base-command');
44
const CliRequestClient = require('../services/cli-http-client');

0 commit comments

Comments
 (0)