Skip to content

Commit

Permalink
Fix patching @slack/rtm-api (#37)
Browse files Browse the repository at this point in the history
* fix patching

use yarn patch methods as using patch-package is broken

* remove patch-package references

* bump yarn

---------

Co-authored-by: Kishan Bagaria <[email protected]>
  • Loading branch information
1Conan and KishanBagaria authored Sep 12, 2023
1 parent 53a7d73 commit f88790d
Show file tree
Hide file tree
Showing 5 changed files with 312 additions and 475 deletions.
561 changes: 281 additions & 280 deletions .yarn/releases/yarn-3.5.0.cjs → .yarn/releases/yarn-3.6.3.cjs

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion .yarnrc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ enableGlobalCache: true

nodeLinker: node-modules

yarnPath: .yarn/releases/yarn-3.5.0.cjs
yarnPath: .yarn/releases/yarn-3.6.3.cjs
9 changes: 4 additions & 5 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,21 +7,20 @@
"patches/**/*"
],
"scripts": {
"prepack": "patch-package && tsc",
"postinstall": "{ patch-package && tsc } || true",
"prepack": "tsc",
"postinstall": "tsc || true",
"build": "yarn tsc",
"lint": "eslint src --ext ts,tsx,js,jsx --cache",
"test": "jest"
},
"dependencies": {
"@slack/logger": "^3.0.0",
"@slack/rtm-api": "^6.1.0",
"@slack/rtm-api": "patch:@slack/rtm-api@^6.1.0#patches/@slack+rtm-api+6.0.0.patch",
"@slack/web-api": "^6.8.1",
"@textshq/platform-sdk": "https://github.com/TextsHQ/platform-sdk#bb853b092ef989ef74d36576e01894d81a3c389a",
"eventemitter3": "^3.1.0",
"finity": "^0.5.4",
"lodash": "^4.17.21",
"patch-package": "^7.0.0",
"postinstall-postinstall": "^2.1.0",
"tough-cookie": "^4.1.2"
},
Expand All @@ -35,7 +34,7 @@
"jest": "^29.5.0",
"typescript": "^5.0.4"
},
"packageManager": "yarn@3.5.0",
"packageManager": "yarn@3.6.3",
"resolutions": {
"@slack/web-api@^6.8.1": "patch:@slack/web-api@npm%3A6.8.1#./.yarn/patches/@slack-web-api-npm-6.8.1-553b0045dc.patch",
"@slack/web-api@^5.3.0": "patch:@slack/web-api@npm%3A6.8.1#./.yarn/patches/@slack-web-api-npm-6.8.1-553b0045dc.patch"
Expand Down
8 changes: 4 additions & 4 deletions patches/@slack+rtm-api+6.0.0.patch
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
diff --git a/node_modules/@slack/rtm-api/dist/RTMClient.d.ts b/node_modules/@slack/rtm-api/dist/RTMClient.d.ts
index 149c9be..2446027 100644
--- a/node_modules/@slack/rtm-api/dist/RTMClient.d.ts
+++ b/node_modules/@slack/rtm-api/dist/RTMClient.d.ts
--- a/dist/RTMClient.d.ts
+++ b/dist/RTMClient.d.ts
@@ -2,6 +2,7 @@
import { Agent } from 'http';
import { EventEmitter } from 'eventemitter3';
Expand Down Expand Up @@ -38,8 +38,8 @@ index 149c9be..2446027 100644
logLevel?: LogLevel;
diff --git a/node_modules/@slack/rtm-api/dist/RTMClient.js b/node_modules/@slack/rtm-api/dist/RTMClient.js
index 44e15b2..4296933 100644
--- a/node_modules/@slack/rtm-api/dist/RTMClient.js
+++ b/node_modules/@slack/rtm-api/dist/RTMClient.js
--- a/dist/RTMClient.js
+++ b/dist/RTMClient.js
@@ -20,7 +20,7 @@ const packageJson = require('../package.json'); // tslint:disable-line:no-requir
* messages.
*/
Expand Down
Loading

0 comments on commit f88790d

Please sign in to comment.