Skip to content

Conversation

@EagleoutIce
Copy link
Member

No description provided.

@EagleoutIce EagleoutIce linked an issue Oct 22, 2025 that may be closed by this pull request
2 tasks
@EagleoutIce EagleoutIce requested a review from Copilot October 22, 2025 16:49
Copy link

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 fixes the workflow to correctly generate files with different extensions by parameterizing the file extension in the update_page function. The main issue being addressed is that the linting rule template should be generated as a .yaml file, not as a .yaml.md file.

Key changes:

  • Modified update_page function to accept file extension as a parameter
  • Updated all function calls to explicitly specify file extensions (md or yaml)
  • Removed the incorrectly generated .github/ISSUE_TEMPLATE/linting-rule.yaml.md file

Reviewed Changes

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

File Description
.github/workflows/broken-links-and-wiki.yaml Refactored update_page function to accept file extension parameter and updated all invocations to pass appropriate extensions
.github/ISSUE_TEMPLATE/linting-rule.yaml.md Removed incorrectly generated file with wrong extension

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@EagleoutIce EagleoutIce changed the title fix: do not add md to upadted linting rule template fix: do not add md to updated linting rule template Oct 22, 2025
@EagleoutIce EagleoutIce merged commit bea833a into main Oct 22, 2025
17 checks passed
@EagleoutIce EagleoutIce deleted the 1979-bug-linter-rule-issue-not-up-to-date branch October 22, 2025 17:00
@EagleoutIce
Copy link
Member Author

This pull request is included in v2.6.2 (see Release v2.6.2 (TS-Queries, new Domain Structure, more Reserved Words, removed Lineage Query)).

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.

[Bug]: Linter rule issue not up-to-date

2 participants