Skip to content

Conversation

@ezw2000
Copy link
Collaborator

@ezw2000 ezw2000 commented Nov 8, 2025

No description provided.

ezw2000 and others added 8 commits November 8, 2025 10:26
- Removed beta indication from the Profit 7 release notes in both Dutch and English versions.
- Updated the menu structure to reflect the changes in the Profit 7 release notes title.
- Removed beta indication from the Profit 7 release notes in both Dutch and English versions.
- Updated the menu structure to reflect the changes in the Profit 7 release notes title.
Validation is now done when committing.
@ezw2000 ezw2000 requested review from Heimmot24 and Copilot November 8, 2025 10:22
@ezw2000 ezw2000 enabled auto-merge November 8, 2025 10:23
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 introduces automated tagging and date validation functionality for markdown documentation files. The main changes include a new script for auto-tagging markdown files and enhanced validation that checks both naming conventions and frontmatter dates for modified files.

  • Added automated tagging system that generates up to 6 relevant tags per markdown file based on content analysis
  • Enhanced markdown naming validation script to support git-mode for checking only changed files and validate frontmatter dates
  • Updated all markdown files with new auto-generated tags and current dates (2025-11-08)

Reviewed Changes

Copilot reviewed 64 out of 65 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
scripts/add_tags_to_markdown.py New script that automatically generates and adds relevant tags to markdown files based on content keywords
scripts/markdown-naming-check.py Enhanced validation script with git-mode support and frontmatter date validation for changed files
.github/workflows/markdown-naming-check.yml Removed the GitHub Actions workflow file
menustructures/multilanguage_menu.json Added duplicate menu entry for "Profit 7 Releasenotes (beta)"
markdownpages/profit/**/*.md Updated frontmatter with auto-generated tags and current date (2025-11-08) across all markdown files
API-Documentation.code-workspace Added new VS Code workspace configuration file

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

@ezw2000 ezw2000 requested a review from Copilot November 11, 2025 07:46
Copilot finished reviewing on behalf of ezw2000 November 11, 2025 07:48
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

Copilot reviewed 64 out of 65 changed files in this pull request and generated 7 comments.


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

Copy link

Copilot AI commented Nov 11, 2025

@ezw2000 I've opened a new pull request, #89, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link

Copilot AI commented Nov 11, 2025

@ezw2000 I've opened a new pull request, #90, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link

Copilot AI commented Nov 11, 2025

@ezw2000 I've opened a new pull request, #91, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Collaborator

@plankie033 plankie033 left a comment

Choose a reason for hiding this comment

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

Mooie automatisering en upgrade van de tags. In hoeverre blijft er ruimte voor custom tags die je extra toevoegt? Ga je ook testen of dit het zoekgedrag verbeterd?

@ezw2000 ezw2000 merged commit a5ce2fc into main Nov 11, 2025
5 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.

3 participants