Skip to content
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

[toolset] 🔥 Revert "drop permissions of container to match host system (#8007)" #8013

Merged
merged 3 commits into from
Mar 13, 2023

Conversation

joanhey
Copy link
Contributor

@joanhey joanhey commented Mar 11, 2023

@joanhey
Copy link
Contributor Author

joanhey commented Mar 11, 2023

Perhaps we need to add dockerfile (tfb) and tfb to GH actions, to run the tests.

@joanhey joanhey changed the title [toolset] Revert "drop permissions of container to match host system (#8007)" [toolset] 🔥 Revert "drop permissions of container to match host system (#8007)" Mar 11, 2023
@joanhey joanhey mentioned this pull request Mar 11, 2023
@NateBrady23
Copy link
Member

@joanhey good point. for some reason, i thought we were doing it that way and that's why i accepted those PRs. good catch

@NateBrady23 NateBrady23 merged commit 918d5ff into TechEmpower:master Mar 13, 2023
@joanhey joanhey deleted the revert-8007 branch March 13, 2023 15:31
@gdams
Copy link
Contributor

gdams commented Mar 13, 2023

What can I do to get these changes back in? More than happy to diagnose the issues

franz1981 pushed a commit to franz1981/FrameworkBenchmarks that referenced this pull request Jun 23, 2023
TechEmpower#8007)" (TechEmpower#8013)

* Revert "drop permissions of container to match host system (TechEmpower#8007)"

This reverts commit 1cf4924.

* [ci lang-only php] To test CI

* [ci run-all] To test CI
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