Skip to content

[Filesystem] make sure temp files can be cleaned up on Windows #58185

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Sep 5, 2024

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Issues
License MIT

based on the findings while debugging the AppVeyor failures that we worked around in #58152

@xabbuh xabbuh force-pushed the filesystem-dump-permissions branch from d7e4e79 to a832b67 Compare September 11, 2024 13:46
@nicolas-grekas
Copy link
Member

Good catch, thanks @xabbuh.

@nicolas-grekas nicolas-grekas merged commit 56fa4dc into symfony:5.4 Sep 16, 2024
10 of 12 checks passed
@xabbuh xabbuh deleted the filesystem-dump-permissions branch September 16, 2024 14:54
nicolas-grekas added a commit that referenced this pull request Sep 18, 2024
This PR was merged into the 7.2 branch.

Discussion
----------

[Filesystem] revert test changes

| Q             | A
| ------------- | ---
| Branch?       | 7.2
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        |
| License       | MIT

we can revert #58152 now that the underlying bug was fixed with #58185

Commits
-------

5ea44be revert test changes
This was referenced Sep 21, 2024
github-merge-queue bot pushed a commit to Lendable/composer-license-checker that referenced this pull request Sep 23, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[phpstan/phpstan-strict-rules](https://redirect.github.com/phpstan/phpstan-strict-rules)
| `1.6.0` -> `1.6.1` |
[![age](https://developer.mend.io/api/mc/badges/age/packagist/phpstan%2fphpstan-strict-rules/1.6.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/packagist/phpstan%2fphpstan-strict-rules/1.6.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/packagist/phpstan%2fphpstan-strict-rules/1.6.0/1.6.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/packagist/phpstan%2fphpstan-strict-rules/1.6.0/1.6.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
| [symfony/filesystem](https://symfony.com)
([source](https://redirect.github.com/symfony/filesystem)) | `6.4.9` ->
`6.4.12` |
[![age](https://developer.mend.io/api/mc/badges/age/packagist/symfony%2ffilesystem/6.4.12?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/packagist/symfony%2ffilesystem/6.4.12?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/packagist/symfony%2ffilesystem/6.4.9/6.4.12?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/packagist/symfony%2ffilesystem/6.4.9/6.4.12?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>phpstan/phpstan-strict-rules
(phpstan/phpstan-strict-rules)</summary>

###
[`v1.6.1`](https://redirect.github.com/phpstan/phpstan-strict-rules/releases/tag/1.6.1)

[Compare
Source](https://redirect.github.com/phpstan/phpstan-strict-rules/compare/1.6.0...1.6.1)

-
[daeec74](http://github.com/phpstan/phpstan-strict-rules/commit/daeec748b53de80a97498462513066834ec28f8b)
- Check if treatPhpDocTypesAsCertain tip is enabled
-
[876574c](http://github.com/phpstan/phpstan-strict-rules/commit/876574c426cfc6531ef26c799d20f0cb97d82f48)
- Fix build
-
[a567f7b](http://github.com/phpstan/phpstan-strict-rules/commit/a567f7b0e5dfe524cc84f177fe5ca2b81f49b92a)
- Pin build-cs
-
[2462e57](http://github.com/phpstan/phpstan-strict-rules/commit/2462e578b2f9b89d0774d284d139d2cd702729bb)
- Test newer PHP versions

</details>

<details>
<summary>symfony/filesystem (symfony/filesystem)</summary>

###
[`v6.4.12`](https://redirect.github.com/symfony/filesystem/releases/tag/v6.4.12)

[Compare
Source](https://redirect.github.com/symfony/filesystem/compare/v6.4.9...v6.4.12)

**Changelog**
(symfony/filesystem@v6.4.11...v6.4.12)

- bug
[symfony/symfony#58185](https://redirect.github.com/symfony/symfony/issues/58185)
\[Filesystem] make sure temp files can be cleaned up on Windows
([@&#8203;xabbuh](https://redirect.github.com/xabbuh))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config
help](https://redirect.github.com/renovatebot/renovate/discussions) if
that's undesired.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/Lendable/composer-license-checker).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44MC4wIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants