Skip to content

Conversation

@GromNaN
Copy link
Member

@GromNaN GromNaN commented Sep 10, 2025

Missing from #3348

@GromNaN GromNaN requested a review from a team as a code owner September 10, 2025 11:15
Copy link

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 removes an unused dependency by cleaning up the symfony/http-foundation package from the composer.json requirements and removing an outdated Symfony copyright header from a session handler file.

  • Removes symfony/http-foundation dependency from composer.json
  • Removes Symfony copyright header from MongoDbSessionHandler.php

Reviewed Changes

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

File Description
composer.json Removes unused symfony/http-foundation dependency
src/Session/MongoDbSessionHandler.php Removes outdated Symfony copyright header

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@GromNaN GromNaN merged commit cee14c4 into mongodb:5.x Sep 11, 2025
72 checks passed
@GromNaN GromNaN deleted the symfony-dep branch September 11, 2025 11:46
GromNaN added a commit to GromNaN/laravel-mongodb-fork that referenced this pull request Dec 5, 2025
GromNaN added a commit to GromNaN/laravel-mongodb-fork that referenced this pull request Dec 8, 2025
GromNaN added a commit that referenced this pull request Dec 8, 2025
* Improve change log categories (#3442)

So that anything labeled `feature` is added to the Features category and docs changes are listed as a separate category

* Bump actions/labeler from 5 to 6 (#3445)

Bumps [actions/labeler](https://github.com/actions/labeler) from 5 to 6.
- [Release notes](https://github.com/actions/labeler/releases)
- [Commits](actions/labeler@v5...v6)

---
updated-dependencies:
- dependency-name: actions/labeler
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Remove explicit dependency to symfony/http-kernel as it's not used directly (#3446)

* Bump mongodb-labs/drivers-github-tools from 2 to 3 (#3452)

Bumps [mongodb-labs/drivers-github-tools](https://github.com/mongodb-labs/drivers-github-tools) from 2 to 3.
- [Release notes](https://github.com/mongodb-labs/drivers-github-tools/releases)
- [Commits](mongodb-labs/drivers-github-tools@v2...v3)

---
updated-dependencies:
- dependency-name: mongodb-labs/drivers-github-tools
  dependency-version: '3'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump github/codeql-action from 3 to 4 (#3455)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3 to 4.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@v3...v4)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-version: '4'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Test with PHP 8.5 and reduce test matrix

* Test all server versions with the lowest PHP and latest Laravel

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Pauline Vos <pvos88@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@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