Update testing-a-pull-request.md

[rosiel]

Purpose / why

After @DonRichards 's comments on Slack I went and added to the docs - I didn't see #2335 .

My preference for mine is that Composer is not supposed to be run as root, and nginx is not the only possible owner for the site files.

I also made a separate section for changes to configs, since they're relevant regardless of which method you choose.

And I added warnings at the top of the page about the file permission issue (which is a real issue, I don't want to diminish it)

What changes were made?

Listed three ways to update configs, one copied from @DonRichards

Verification

Are the instructions readable and typo- or error-free?

Interested Parties

• @Islandora/documentation
• @Islandora/committers

Checklist

Pull-request Reviewer

Pull-request reviewer should ensure the following:

• Does this PR link to related issues?
• Does the proposed documentation align with the Islandora Documentation Style Guide?
• Are the changes accurate, useful, free of typos, etc?

Person Merging

The person merging should ensure the following:

• Does mkdocs still build successfully? (This is indicated by TravisCI passing. To test locally, and see warnings, see How To Build Documentation.)
• If pages are renamed or removed, have all internal links to those pages been fixed?
• If pages are added, have they been linked to or placed in the menu?
• Did the PR receive at least one approval from a committer, and all issues raised have been addressed?

[github-actions]

PR Preview Action v1.4.7
🚀 Deployed preview to https://Islandora.github.io/documentation/pr-preview/pr-2336/
on branch gh-pages at 2024-07-29 13:34 UTC