Skip to content

Fix devcontainer configuration and update dependencies#942

Merged
decyjphr merged 4 commits intodependabot/npm_and_yarn/probot-14.2.4from
decyjphr/fix-devcontainer
Mar 3, 2026
Merged

Fix devcontainer configuration and update dependencies#942
decyjphr merged 4 commits intodependabot/npm_and_yarn/probot-14.2.4from
decyjphr/fix-devcontainer

Conversation

@decyjphr
Copy link
Collaborator

@decyjphr decyjphr commented Mar 3, 2026

Update the Node.js version in the devcontainer configuration and adjust the installation script for the AWS SAM CLI. Also, update the minimatch dependency version in the package lock file.

Copilot AI review requested due to automatic review settings March 3, 2026 13:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the development container environment to Node.js 22 and adjusts tooling installation in the devcontainer image, along with aligning the lockfile dependency spec for minimatch.

Changes:

  • Bump devcontainer Node.js variant from 20-bookworm to 22-bookworm.
  • Change AWS SAM CLI download logic in the devcontainer Dockerfile to use uname -m for architecture selection.
  • Update minimatch version spec in package-lock.json to ^10.2.1.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File Description
package-lock.json Updates the minimatch dependency spec recorded in the lockfile.
.devcontainer/devcontainer.json Switches the devcontainer build ARG to 22-bookworm.
.devcontainer/Dockerfile Switches base image variant to Node 22 and modifies SAM CLI installation architecture detection.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

decyjphr and others added 3 commits March 3, 2026 08:38
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@decyjphr decyjphr merged commit 6eb1718 into dependabot/npm_and_yarn/probot-14.2.4 Mar 3, 2026
2 checks passed
decyjphr added a commit that referenced this pull request Mar 3, 2026
* build(deps): bump probot from 13.4.4 to 14.2.4

Bumps [probot](https://github.com/probot/probot) from 13.4.4 to 14.2.4.
- [Release notes](https://github.com/probot/probot/releases)
- [Commits](probot/probot@v13.4.4...v14.2.4)

---
updated-dependencies:
- dependency-name: probot
  dependency-version: 14.2.4
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* Fix test suite failures caused by ESM-only `@octokit/core` and `probot` dependencies (#938)

* Initial plan

* Initial plan for fixing failing test suites

Co-authored-by: decyjphr <57544838+decyjphr@users.noreply.github.com>

* Fix failing test suites caused by ESM-only dependencies

Co-authored-by: decyjphr <57544838+decyjphr@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: decyjphr <57544838+decyjphr@users.noreply.github.com>

* Update minimatch dependency version to 10.2.1

* Fix devcontainer configuration and update dependencies (#942)

* fix devcontainer

* Update .devcontainer/Dockerfile

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update .devcontainer/Dockerfile

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update .devcontainer/devcontainer.json

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Yadhav Jayaraman <57544838+decyjphr@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

2 participants