Skip to content

Origin/actions auto#288

Merged
danielmarv merged 2 commits intonext-js-migrationfrom
origin/actions-auto
Feb 24, 2026
Merged

Origin/actions auto#288
danielmarv merged 2 commits intonext-js-migrationfrom
origin/actions-auto

Conversation

@danielmarv
Copy link
Member

This pull request introduces a comprehensive contributor documentation hub and updates the project structure references for clarity and onboarding. The main focus is to streamline the onboarding process, clarify content folder usage, and provide step-by-step guides for page addition, contribution, testing, and GitHub automation. Legacy Hugo references are removed, and new docs are added to guide contributors through every aspect of the workflow.

Documentation and onboarding improvements:

  • Added a new docs/README.md as the central documentation hub, with a recommended reading order and quick commands for contributors.
  • Introduced detailed guides: repository overview (docs/01-repo-overview.md), content folder usage (docs/02-content-folder.md), adding pages (docs/03-adding-pages.md), first contribution checklist (docs/04-first-contribution-checklist.md), testing and quality checks (docs/05-testing-and-quality-checks.md), and GitHub automation (docs/06-github-automation.md). [1] [2] [3] [4] [5] [6]

Project structure and content folder clarification:

  • Updated README.md to remove legacy Hugo references, clarify the role of the content/ folder, and highlight the new docs hub as the first stop for contributors. [1] [2] [3]
  • Added guidance in ADDING_PAGES.md to direct first-time contributors to the docs hub and content folder guide before proceeding.

Legacy cleanup:

  • Removed the obsolete Hugo configuration file config.toml to reflect the migration to Next.js and markdown content.

…y overview, content guidelines, and contribution checklist

Signed-off-by: Daniel Ntege <[email protected]>
@danielmarv danielmarv merged commit 3276ad2 into next-js-migration Feb 24, 2026
5 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Migration to NextJS Feb 24, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new contributor documentation hub under docs/ and updates existing top-level contributor docs to point to it, while also cleaning up legacy configuration (config.toml) to reflect the current Next.js-based setup.

Changes:

  • Added a structured contributor docs hub (docs/README.md) plus six step-by-step guides covering repo structure, content workflow, testing, and GitHub automation.
  • Updated contributor entry points (README.md, ADDING_PAGES.md) to direct newcomers to the new docs hub and clarify content/ usage.
  • Removed the obsolete Hugo config.toml.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
docs/README.md New documentation hub entry point with reading order and common commands.
docs/01-repo-overview.md New repo overview doc describing architecture and key folders.
docs/02-content-folder.md New guide describing content/ structure conventions and localization rules.
docs/03-adding-pages.md New end-to-end quick process for adding a page (content, routes, assets, checks).
docs/04-first-contribution-checklist.md New checklist for first-time contributors (branching, validation, PR expectations).
docs/05-testing-and-quality-checks.md New contributor-facing testing/linting guidance and manual verification checklist.
docs/06-github-automation.md New overview of issue/PR automation commands, reviewers config, and lifecycle.
README.md Removes legacy Hugo block in “Runtime layers”, clarifies content/, and links to docs hub.
ADDING_PAGES.md Adds prominent pointers to the new docs hub and the content folder guide.
config.toml Deleted obsolete Hugo configuration file.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +23 to +24
- Issues use status labels such as triage/in-progress/blocked/closed
- PRs use draft/in-review/approved/changes-requested/merged/closed
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

The status label names in this doc don't match what the automation actually applies. The status-labels.yml workflow creates/applies labels like status: triage, status: in-progress, status: blocked, etc., and PR labels like status: draft, status: in-review, etc. Update this section to use the exact label strings (including the status: prefix) so contributors can search/filter the right labels.

Suggested change
- Issues use status labels such as triage/in-progress/blocked/closed
- PRs use draft/in-review/approved/changes-requested/merged/closed
- Issues use status labels such as `status: triage`, `status: in-progress`, `status: blocked`, `status: closed`
- PRs use status labels such as `status: draft`, `status: in-review`, `status: approved`, `status: changes-requested`, `status: merged`, `status: closed`

Copilot uses AI. Check for mistakes.
Comment on lines +7 to +10
- Next.js App Router application (primary runtime)
- Markdown content (legacy Hugo format, still actively used for content)
- i18n routing and locale files for English and German
- GitHub Actions automation for issue/PR workflows
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

This file reintroduces a 'legacy Hugo format' reference, but the PR description says legacy Hugo references are being removed and config.toml has been deleted. Either remove the Hugo wording here or rephrase it more precisely (e.g., Markdown content with Hugo-style shortcode compatibility) so the documentation and PR intent are consistent.

Copilot uses AI. Check for mistakes.
Comment on lines 20 to 23

- **Legacy Hugo content (migrating)**
- Markdown content in `content`.
- Hugo templates in `src/layouts`.
- Hugo configuration in `config.toml`.
- **Markdown content**
- Markdown content in `content/`.
- Built static artifacts live in `public` (do not edit manually).
Copy link

Copilot AI Feb 24, 2026

Choose a reason for hiding this comment

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

This updated section frames the repo as purely 'Markdown content', but elsewhere in README.md the intro still states that 'legacy Hugo content' is kept. With config.toml removed, the README should use consistent wording across sections (either remove the Hugo mention in the intro, or clarify that markdown remains Hugo-compatible only in specific ways like shortcode support).

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants