Skip to content

[DOCS] fix pre merge checks#1023

Merged
kblaszczak-intel merged 7 commits intoopen-edge-platform:mainfrom
kblaszczak-intel:DOCS-new-pre-merge-checks
Feb 18, 2026
Merged

[DOCS] fix pre merge checks#1023
kblaszczak-intel merged 7 commits intoopen-edge-platform:mainfrom
kblaszczak-intel:DOCS-new-pre-merge-checks

Conversation

@kblaszczak-intel
Copy link
Copy Markdown
Contributor

@kblaszczak-intel kblaszczak-intel commented Feb 17, 2026

📝 Description

New pre-merge checks for documentation PRs. This check will run when files in the main docs folder are changed. It will use the exact same build setup as the main documentation publishing job, getting the config files from the docs server, so that no credentials are needed.

The check will be triggered in the main docs folder only because we intend it to host docs from microservice folders as well. The individual microservice "docs" folders will include references to it.

We are planning a follow-up PR to add the feature of attaching the documentation html built for the check as a zip artifact. This way it will be possible to preview the changed articles.

✨ Type of Change

Select the type of change your PR introduces:

  • 🐞 Bug fix – Non-breaking change which fixes an issue
  • 🚀 New feature – Non-breaking change which adds functionality
  • 🔨 Refactor – Non-breaking change which refactors the code base
  • 💥 Breaking change – Changes that break existing functionality
  • 📚 Documentation update
  • 🔒 Security update
  • 🧪 Tests
  • 🚂 CI

🧪 Testing Scenarios

Describe how the changes were tested and how reviewers can test them too:

  • ✅ Tested manually
  • 🤖 Ran automated end-to-end tests

✅ Checklist

Before submitting the PR, ensure the following:

  • 🔍 PR title is clear and descriptive
  • 📝 For internal contributors: If applicable, include the JIRA ticket number (e.g., ITEP-123456) in the PR title. Do not include full URLs
  • 💬 I have commented my code, especially in hard-to-understand areas
  • 📄 I have made corresponding changes to the documentation
  • ✅ I have added tests that prove my fix is effective or my feature works

@kblaszczak-intel kblaszczak-intel marked this pull request as draft February 17, 2026 08:43
Comment thread .github/workflows/documentation-check.yaml Fixed
Comment thread .github/workflows/documentation-check.yaml Fixed
@kblaszczak-intel kblaszczak-intel marked this pull request as ready for review February 17, 2026 09:33
saratpoluri
saratpoluri previously approved these changes Feb 17, 2026
Comment thread .github/workflows/documentation-check.yaml Outdated
@kblaszczak-intel kblaszczak-intel dismissed jouillet’s stale review February 18, 2026 06:43

Thanks for noticing that! The fix is in place, fingers crossed it all works :)

@kblaszczak-intel kblaszczak-intel enabled auto-merge (squash) February 18, 2026 10:00
@kblaszczak-intel kblaszczak-intel merged commit 3b3708f into open-edge-platform:main Feb 18, 2026
23 checks passed
@kblaszczak-intel kblaszczak-intel deleted the DOCS-new-pre-merge-checks branch March 9, 2026 06:33
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.

6 participants