Skip to content

docs: replace template README with CV repo overview - #76

Merged
DanBennettUK merged 1 commit into
masterfrom
docs/readme-overhaul
Aug 9, 2026
Merged

docs: replace template README with CV repo overview#76
DanBennettUK merged 1 commit into
masterfrom
docs/readme-overhaul

Conversation

@DanBennettUK

Copy link
Copy Markdown
Owner

Replaces the sproogen modern-resume-theme template README (which still points at the theme author's email, buy-me-a-coffee link and GitHub Pages setup) with a short overview of this repo: what it is, content structure, local dev and Vercel deployment.

No content or config changes.

@vercel

vercel Bot commented Aug 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dan-bennett-online-cv Ready Ready Preview Aug 9, 2026 11:09pm

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ea41fd8094

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md
## Deployment

If you have docker installed you can simply run `docker-compose up` to launch the site in a container, it will then be hosted at `http://localhost:4000`
Hosted on Vercel. Merging to `master` triggers a production deployment; PR branches get preview deployments.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use main in the deployment instructions

The repository's active base branch is main (refs/heads/main points at this commit's parent), and there is no master ref. As written, maintainers following this deployment guidance would target a nonexistent branch rather than the branch used by the repository, so the documented production-deployment flow cannot work; update this to name main (or the actual Vercel production branch if it is configured differently).

Useful? React with 👍 / 👎.

@DanBennettUK
DanBennettUK merged commit bb2a02a into master Aug 9, 2026
7 checks passed
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.

1 participant