Release script corrections

[smillst]

No description provided.

[coderabbitai]

📝 Walkthrough

Walkthrough

The release scripts were changed in two places. In docs/developer/release/release_build.py, ensure_user_access was added to imports; calls to ensure_group_access and ensure_user_access were added for the dev website directory before copying, and shutil.copytree was changed to use dirs_exist_ok=True. In docs/developer/release/release_utils.py, commit_tag_and_push now removes the pre-commit hook at .git/hooks/pre-commit before checking for changes, committing, tagging, and pushing.

🚥 Pre-merge checks | ✅ 1

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@docs/developer/release/release_utils.py`:
- Around line 292-294: The remove-precommit command should use the -f flag and
not an f-string to avoid failing when the hook is missing and to eliminate the
unnecessary f prefix; update the call to execute that currently invokes
execute(f"rm .git/hooks/pre-commit", working_dir=path) so it instead calls
execute("rm -f .git/hooks/pre-commit", working_dir=path) (locate the invocation
of execute in release_utils.py that removes .git/hooks/pre-commit).