Skip to content

Use sudo explicitly for caddy commands.#86

Open
rowan-m wants to merge 2 commits intomainfrom
caddy-sudo
Open

Use sudo explicitly for caddy commands.#86
rowan-m wants to merge 2 commits intomainfrom
caddy-sudo

Conversation

@rowan-m
Copy link
Copy Markdown
Collaborator

@rowan-m rowan-m commented Mar 11, 2026

I have npm installed on my default user account via nvm and so it's not available globally. This means running sudo npm run dev:local fails.

This PR updates caddy.sh to just use sudo on the caddy commands pointing at the locally installed version.

I'm not sure if this makes sense as a project change, but it does let me run the development environment on my machine.

@agektmr agektmr requested review from agektmr and ewewraw March 16, 2026 08:16
@agektmr
Copy link
Copy Markdown
Collaborator

agektmr commented Apr 1, 2026

Verified that this works but could you also update README with your tips?

@rowan-m
Copy link
Copy Markdown
Collaborator Author

rowan-m commented Apr 1, 2026

Verified that this works but could you also update README with your tips?

@agektmr updated README.md with details and some clean up.

@agektmr
Copy link
Copy Markdown
Collaborator

agektmr commented Apr 2, 2026

Can you also write about the flags?

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