Skip to content

fix: Octokit usage#399

Merged
EthanFreestone merged 2 commits intomainfrom
fix/octokit
Apr 15, 2026
Merged

fix: Octokit usage#399
EthanFreestone merged 2 commits intomainfrom
fix/octokit

Conversation

@EthanFreestone
Copy link
Copy Markdown
Contributor

Octokit is now an ESM module as of v21.0.0. stripes-cli has updated to v22 as of this PR

Brings the import into ESM patterns

Octokit is now an ESM module as of v21.0.0. stripes-cli has updated to v22 as of [this PR](#395)

Brings the import into ESM patterns
@EthanFreestone EthanFreestone requested a review from a team as a code owner April 15, 2026 10:15
@EthanFreestone EthanFreestone marked this pull request as draft April 15, 2026 10:15
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 15, 2026

Jest Unit Test Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit b15ee4e. ± Comparison against base commit 96f7875.

♻️ This comment has been updated with latest results.

Comment thread lib/environment/inventory.js Outdated
Copy link
Copy Markdown
Contributor

@JohnC-80 JohnC-80 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Please removed the commented code.

@EthanFreestone EthanFreestone marked this pull request as ready for review April 15, 2026 14:37
@sonarqubecloud
Copy link
Copy Markdown

@EthanFreestone
Copy link
Copy Markdown
Contributor Author

Commented code removed, merging

@EthanFreestone EthanFreestone merged commit d145b29 into main Apr 15, 2026
15 checks passed
@EthanFreestone EthanFreestone deleted the fix/octokit branch April 15, 2026 14:40
JohnC-80 pushed a commit that referenced this pull request Apr 17, 2026
Octokit is now an ESM module as of v21.0.0. stripes-cli has updated to
v22 as of [this PR](#395)

Brings the import into ESM patterns
JohnC-80 pushed a commit that referenced this pull request Apr 17, 2026
Octokit is now an ESM module as of v21.0.0. stripes-cli has updated to
v22 as of [this PR](#395)

Brings the import into ESM patterns
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants