Skip to content

fix: remove a conditional that prevented image url updates for repositories created from private repository templates#125

Merged
FidelusAleksander merged 1 commit intomainfrom
replace-imgs-in-private-repos
Mar 24, 2026
Merged

fix: remove a conditional that prevented image url updates for repositories created from private repository templates#125
FidelusAleksander merged 1 commit intomainfrom
replace-imgs-in-private-repos

Conversation

@FidelusAleksander
Copy link
Copy Markdown
Contributor

This pull request makes a minor update to the workflow configuration by removing a conditional check from the "Find and replace image URLs in step files" job step. This simplifies the workflow and ensures that the step always runs, regardless of the template repository output.

Changes

Checklist

  • I have added or updated appropriate labels to this PR
  • I have tested my changes
  • I have updated the documentation if needed

Removed conditional check for template repository before finding and replacing image URLs.
Copilot AI review requested due to automatic review settings March 20, 2026 08:26
@github-actions github-actions bot added the workflows Changes to reusable workflows label Mar 20, 2026
Copy link
Copy Markdown
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

Removes a workflow step guard so that the image URL replacement in .github/steps/*.md always runs, ensuring repositories created from private templates still get updated image links.

Changes:

  • Removed the if: condition that skipped the “Find and replace image URLs in step files” step when TEMPLATE_REPOSITORY output was empty.

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

Copy link
Copy Markdown

@Lilili14333 Lilili14333 left a comment

Choose a reason for hiding this comment

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

Changes

Checklist

I have added or updated appropriate labels to this PR
I have tested my changes
I have updated the documentation if needed

@FidelusAleksander FidelusAleksander requested a review from a team March 23, 2026 12:58
@FidelusAleksander FidelusAleksander merged commit 1708f86 into main Mar 24, 2026
10 checks passed
@FidelusAleksander FidelusAleksander deleted the replace-imgs-in-private-repos branch March 24, 2026 08:08
@Lilili14333
Copy link
Copy Markdown

🥰🥰🥰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflows Changes to reusable workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants