Commit 6cc49d9
committed
fix(ci): rename the cross-repo client — github-script already declares
`actions/github-script` injects an `octokit` binding into the script scope, so
`const octokit = ...` aborts the run before any logic executes:
SyntaxError: Identifier 'octokit' has already been declared
Caught on objectstack#4553's own merge — the first time the workflow ever ran.
The pre-merge `node --check` missed it because the test wrapper declared only
{github, context, core, require}; a wrapper that omits an injected identifier
cannot see a collision with it. The wrapper now carries the full injected set,
and the old spelling fails that check.octokit
1 parent 76515a5 commit 6cc49d9
1 file changed
Lines changed: 4 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
105 | 105 | | |
106 | 106 | | |
107 | 107 | | |
108 | | - | |
| 108 | + | |
109 | 109 | | |
110 | 110 | | |
111 | 111 | | |
112 | | - | |
| 112 | + | |
113 | 113 | | |
114 | 114 | | |
115 | 115 | | |
116 | 116 | | |
117 | 117 | | |
118 | 118 | | |
119 | | - | |
| 119 | + | |
120 | 120 | | |
121 | 121 | | |
122 | 122 | | |
123 | 123 | | |
124 | 124 | | |
125 | 125 | | |
126 | | - | |
| 126 | + | |
127 | 127 | | |
128 | 128 | | |
129 | 129 | | |
| |||
0 commit comments