Skip to content

Conversation

@dawnkelly09
Copy link
Collaborator

📝 Description

Provide a clear and concise description of your changes.

Removes the last modified lines from site-index.json outputs to reduce excessive merge conflicts.

🔍 Review Preference

Choose one:

  • ✅ I have time to handle formatting/style feedback myself
  • ⚡ Docs team handles formatting (check "Allow edits from maintainers")

🤖 AI-Ready Docs

If content changed, regenerate AI files:

  • ✅ I ran python3 scripts/generate_llms.py
  • ⚡ Docs team will regenerate (check "Allow edits from maintainers")

✅ Checklist

@dawnkelly09 dawnkelly09 added B0 - Needs Review Pull request is ready for review A3 - Quick Fixes Pull request contains small fixes (i.e., fixing a typo, removing old images, etc.) labels Oct 24, 2025
@dawnkelly09 dawnkelly09 marked this pull request as ready for review October 24, 2025 16:58
@dawnkelly09 dawnkelly09 requested a review from a team as a code owner October 24, 2025 16:58
Copilot AI review requested due to automatic review settings October 24, 2025 16:58
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 removes the last_modified field from the site-index.json file to reduce merge conflicts that occur when multiple contributors update documentation simultaneously. The change modifies the generation script to exclude modification timestamps from the output, resulting in a cleaner generated JSON file that only changes when actual content is updated.

  • Removes last_modified timestamp field from site index generation logic
  • Updates generated site-index.json to remove all existing last_modified entries

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
scripts/generate_site_index.py Removes line that adds last_modified field to page metadata dictionary
.ai/site-index.json Removes all last_modified timestamp entries from generated site index

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

Copy link
Collaborator

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

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

Thank you! 🙏

@eshaben eshaben added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Oct 27, 2025
@eshaben eshaben merged commit 552e509 into master Oct 29, 2025
4 checks passed
@eshaben eshaben deleted the dawn/remove-last-mod-field branch October 29, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A3 - Quick Fixes Pull request contains small fixes (i.e., fixing a typo, removing old images, etc.) B1 - Ready to Merge Pull request is ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants