Skip to content

Make build.sh executable#3265

Open
MTRNord wants to merge 3 commits intomainfrom
MTRNord/fix-perms
Open

Make build.sh executable#3265
MTRNord wants to merge 3 commits intomainfrom
MTRNord/fix-perms

Conversation

@MTRNord
Copy link
Contributor

@MTRNord MTRNord commented Mar 6, 2026

Description

Does what the title says. It annoyed me :D

Related issues

Role

Website & Content WG

Timeline

Signoff

Commits!

Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
@MTRNord MTRNord requested a review from a team as a code owner March 6, 2026 17:51
@MTRNord MTRNord added the enhancement This would be an improvement to the website. label Mar 6, 2026
@HarHarLinks
Copy link
Collaborator

Why is this even needed, and why does the shebang contain a space?

@MTRNord
Copy link
Contributor Author

MTRNord commented Mar 6, 2026

Why is this even needed, and why does the shebang contain a space?

I have no idea :/ I am guessing cloudflare uses it? I dont know really. I just saw it and got annoyed by it not having bee 755 yet.

@MTRNord
Copy link
Contributor Author

MTRNord commented Mar 6, 2026

The shebang being broken probably always was a bug but doesnt matter the way cloudflare calls that file?

Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 6, 2026

Deploying matrix-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: c7c05b1
Status: ✅  Deploy successful!
Preview URL: https://09f7695a.matrix-website.pages.dev
Branch Preview URL: https://mtrnord-fix-perms.matrix-website.pages.dev

View logs

@HarHarLinks HarHarLinks requested a review from a team March 7, 2026 09:40
Co-authored-by: Michael Weimann <weeman@frankfurt.ccc.de>
Signed-off-by: Marcel <MTRNord@users.noreply.github.com>
@HarHarLinks
Copy link
Collaborator

(I suggest not to merge this without anyone who actually touches cloudflare in the loop about it)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement This would be an improvement to the website.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants