Skip to content

chore(gha): update mobsuccess.yml GitHub workflow - #61

Open
ms-bot wants to merge 1 commit into
masterfrom
mobsuccessbot/workflow-mobsuccess
Open

chore(gha): update mobsuccess.yml GitHub workflow#61
ms-bot wants to merge 1 commit into
masterfrom
mobsuccessbot/workflow-mobsuccess

Conversation

@ms-bot

@ms-bot ms-bot commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

Why is this needed?

This pull request has been created by a robot to update the mobsuccess.yml GitHub workflow.

@ms-bot ms-bot added the mobsuccessbot Pull requests that enforce company policies label Jan 23, 2026
@ms-bot
ms-bot force-pushed the mobsuccessbot/workflow-mobsuccess branch from e39c3b6 to 74049c4 Compare July 19, 2026 08:49
Copilot AI lite review requested due to automatic review settings August 4, 2026 07:57
@ms-bot
ms-bot force-pushed the mobsuccessbot/workflow-mobsuccess branch from 74049c4 to 53fbf8a Compare August 4, 2026 07:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the mobsuccess.yml GitHub Actions workflow used to enforce Mobsuccess PR compliance checks, including adding additional PR-safety validation logic.

Changes:

  • Switches the job runner label from ubuntu-24.04 to ubuntu-slim.
  • Adds a PR-only checkout step using actions/checkout.
  • Adds a PR-only guard script to detect “ignored directory replacement” cases and disallow repo-escaping symlinks.

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

MobsuccessPRCompliance:
name: "Mobsuccess PR Compliance"
runs-on: ubuntu-24.04
runs-on: ubuntu-slim
Comment on lines +102 to +105
ignored() {
git -c core.excludesFile=/dev/null -C "$sandbox" \
check-ignore -q --no-index -- "$1"
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mobsuccessbot Pull requests that enforce company policies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants