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

Fix: Strauss directory permissions #7647

Closed
wants to merge 1 commit into from

Conversation

defunctl
Copy link

@defunctl defunctl commented Dec 11, 2024

Description

I have A PR into Strauss to fix the permissions here: BrianHenryIE/strauss#130

Until that's merged and released, this should work to fix things in your workflows.

Successful Job Here: https://github.com/impress-org/givewp/actions/runs/12285568512

@defunctl defunctl added the type: bug Existing functionality is broken label Dec 11, 2024
@defunctl defunctl self-assigned this Dec 11, 2024
Copy link
Contributor

@JasonTheAdams JasonTheAdams left a comment

Choose a reason for hiding this comment

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

Appreciate the workaround, @defunctl!

Let's hold on merging this for a day or two to see if Brian accepts and merges your fix in Strauss. If it's looking like it may take some time, we'll merge.

@defunctl
Copy link
Author

defunctl commented Dec 11, 2024

@JasonTheAdams no worries, you'll definitely want to add this to your existing PR if you don't merge this one.

image

@jonwaldstein
Copy link
Contributor

@defunctl thank you!!! 😭

@jonwaldstein
Copy link
Contributor

oh wait it was already merged and released in 0.20.1 🙌

@defunctl does this mean we can just use that version and add the dump-autoload line without this PR?

@defunctl
Copy link
Author

oh wait it was already merged and released in 0.20.1 🙌

@defunctl does this mean we can just use that version and add the dump-autoload line without this PR?

@jonwaldstein exactly!

@JasonTheAdams
Copy link
Contributor

Excited for the quick turnaround! Thanks for submitting the Strauss fix, @defunctl! 🌮

Closing in favor of the new fix direction.

@defunctl defunctl deleted the fix/strauss branch December 12, 2024 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Existing functionality is broken
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants