Skip to content

Update probe-engineer.yaml #6590

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Update probe-engineer.yaml #6590

wants to merge 12 commits into from

Conversation

letzya
Copy link
Collaborator

@letzya letzya commented Jun 25, 2025

User description

Preview Link

Checklist

  • Added a preview link
  • Reviewed AI PR Agent suggestions
  • For Tyk Members - Added Jira DX PR ticket to the subject
  • For Tyk Members - Added the appropriate release labels (for fixes add the latest release)

New Contributors


PR Type

enhancement


Description

  • Extend workflow triggers to issues and pull requests

  • Add support for 'opened' events on issues and PRs

  • Improve automation coverage for engineering probe handler

  • No changes to workflow logic or permissions


Changes walkthrough 📝

Relevant files
Enhancement
probe-engineer.yaml
Add issue and PR triggers to workflow                                       

.github/workflows/probe-engineer.yaml

  • Added triggers for 'opened' events on issues and pull requests
  • Expanded workflow activation beyond issue comments
  • No changes to permissions or main workflow logic
  • +5/-1     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • Copy link
    Contributor

    ⚠️ Deploy preview for PR #6590 did not become live after 3 attempts.
    Please check Netlify or try manually: Preview URL

    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Workflow Trigger Expansion

    The workflow now triggers on 'opened' events for both issues and pull requests. Ensure that the workflow logic is compatible with these new triggers and does not perform unintended actions when these events occur.

    issues:
      types: [opened] #[opened, labeled]
    pull_request:
      types: [opened] #[opened , labeled]

    Copy link
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @letzya letzya requested review from buger and yurisasuke June 25, 2025 11:38
    Copy link

    netlify bot commented Jun 25, 2025

    PS. Add to the end of url /docs/nightly

    Name Link
    🔨 Latest commit 327889a
    🔍 Latest deploy log https://app.netlify.com/projects/tyk-docs/deploys/685bf3ad36f15b0008b9b622
    😎 Deploy Preview https://deploy-preview-6590--tyk-docs.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify project configuration.

    Copy link

    netlify bot commented Jun 25, 2025

    PS. Add to the end of url /docs/nightly

    Name Link
    🔨 Latest commit 1d37f04
    🔍 Latest deploy log https://app.netlify.com/projects/tyk-docs/deploys/6867d51086698300089024cb
    😎 Deploy Preview https://deploy-preview-6590--tyk-docs.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify project configuration.

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

    Successfully merging this pull request may close these issues.

    3 participants