Skip to content

feat: modernize repo for Pi-hole v6 - #3

Merged
TimInTech merged 4 commits into
mainfrom
chore/modernize-repo-v6
Sep 5, 2026
Merged

TimInTech merged 4 commits into
mainfrom
chore/modernize-repo-v6

Conversation

@TimInTech

Copy link
Copy Markdown
Owner

Resolves #2. Updates READMEs, adds docker-compose with FTLCONF, and adds missing files.

@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: 024544ebee

ℹ️ 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
Comment on lines +46 to +47
1. Clone the repo or copy `docker-compose.yml` and `.env`.
2. Edit `.env` to set your secure password and timezone.

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 Include the environment template used by the quickstart

The documented .env file is absent from the commit (and the existing *.env ignore rule would prevent adding it normally), while docker-compose.yml obtains PIHOLE_PASSWORD from that file. A user following these steps therefore cannot edit the promised template, and starting the stack as written leaves the configured API password value undefined. Track a non-secret template such as .env.example and update these instructions, or document the required variables explicitly.

Useful? React with 👍 / 👎.

@TimInTech
TimInTech merged commit 4fc3ffa into main Sep 5, 2026
7 checks passed
@TimInTech
TimInTech deleted the chore/modernize-repo-v6 branch September 5, 2026 19:36
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.

Modernize Pi-hole-v6 Repository

1 participant