Skip to content

Conversation

@m-y-mo
Copy link
Contributor

@m-y-mo m-y-mo commented Oct 27, 2025

We forgot to add the GITHUB_AUTH_HEADER environment variable in the smoketest, which is needed by the github-official mcp server.

Copilot AI review requested due to automatic review settings October 27, 2025 08:59
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 fixes a broken smoketest by adding the missing GITHUB_AUTH_HEADER environment variable required by the github-official MCP server.

Key Changes:

  • Added GITHUB_AUTH_HEADER environment variable to the smoketest workflow

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

@m-y-mo
Copy link
Contributor Author

m-y-mo commented Oct 27, 2025

smoke test

@github-actions
Copy link

⚠️ Cannot proceed with deployment

  • reviewDecision: REVIEW_REQUIRED
  • commitStatus: PENDING

CI checks must be passing and the PR must be approved in order to continue

@m-y-mo
Copy link
Contributor Author

m-y-mo commented Oct 27, 2025

smoke test

@github-actions
Copy link

⚠️ Cannot proceed with deployment

  • reviewDecision: REVIEW_REQUIRED
  • commitStatus: SUCCESS

CI checks are passing but an approval is required before you can proceed with deployment

@m-y-mo
Copy link
Contributor Author

m-y-mo commented Oct 27, 2025

smoke test

@github-actions
Copy link

Deployment Triggered 🚀

m-y-mo, started a branch deployment to production (branch: fix_smoketest)

You can watch the progress here 🔗

Details
{
  "type": "branch",
  "environment": {
    "name": "production",
    "url": null
  },
  "deployment": {
    "timestamp": "2025-10-27T10:22:46.621Z",
    "logs": "https://github.com/GitHubSecurityLab/seclab-taskflow-agent/actions/runs/18837616699"
  },
  "git": {
    "branch": "fix_smoketest",
    "commit": "96a86548db02a678a2542307ceeec898b82e5195",
    "verified": false,
    "committer": "m-y-mo",
    "html_url": "https://github.com/GitHubSecurityLab/seclab-taskflow-agent/commit/96a86548db02a678a2542307ceeec898b82e5195"
  },
  "context": {
    "actor": "m-y-mo",
    "noop": false,
    "fork": false,
    "comment": {
      "created_at": "2025-10-27T10:22:27Z",
      "updated_at": "2025-10-27T10:22:27Z",
      "body": "smoke test",
      "html_url": "https://github.com/GitHubSecurityLab/seclab-taskflow-agent/pull/38#issuecomment-3450537573"
    }
  },
  "parameters": {
    "raw": null,
    "parsed": null
  }
}

@github-actions
Copy link

Deployment Results ❌

m-y-mo had a failure when deploying branch fix_smoketest to production

Details
{
  "status": "failure",
  "environment": {
    "name": "production",
    "url": null
  },
  "deployment": {
    "id": 3210295459,
    "timestamp": "2025-10-27T10:25:38.785Z",
    "logs": "https://github.com/GitHubSecurityLab/seclab-taskflow-agent/actions/runs/18837616699",
    "duration": 172
  },
  "git": {
    "branch": "fix_smoketest",
    "commit": "96a86548db02a678a2542307ceeec898b82e5195",
    "verified": false
  },
  "context": {
    "actor": "m-y-mo",
    "noop": false,
    "fork": false
  },
  "reviews": {
    "count": 1,
    "decision": "APPROVED"
  },
  "parameters": {
    "raw": null,
    "parsed": null
  }
}

@m-y-mo m-y-mo merged commit 42f08ef into main Oct 27, 2025
5 checks passed
@m-y-mo m-y-mo deleted the fix_smoketest branch October 27, 2025 12:31
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.

3 participants