chore: remove unused workflows and stale CI config#1662
Conversation
no longer used
The workflow stamped response_time_in_business_days into a hidden probot metadata block on each issue. No consumer reads the value.
- .travis.yml: pins Xcode 11.1 / iOS 13.1 from 2019; CI runs on .github/workflows/ci.yml (Xcode 16.4). - issue_template.md: shadowed by .github/ISSUE_TEMPLATE/*.yml form templates, which GitHub prefers. - default: ci.yml writes this file every run, so the committed copy is not load-bearing.
nan-li
changed the title
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
LGTM — straightforward deletion of unused workflows, stale Travis config, and superseded root files.
Extended reasoning...
Overview
This PR is deletion-only, removing 11 files: three Asana sync workflows, two Claude Code workflows, the response-time tracking workflow plus its two JS helpers (os_probot_metadata.js, set_response_times.js), the stale .travis.yml, the root issue_template.md, and the default scheme file. No SDK or production code is touched.
Security risks
None — this PR reduces attack surface. Removing the Asana workflows eliminates the third-party PAT dependency and any associated supply-chain risk. The Claude workflows that used ANTHROPIC_API_KEY are also removed (they're provisioned separately by the platform team per the description). No new secrets, permissions, or auth flows are introduced.
Level of scrutiny
Low. This is mechanical cleanup of CI/repo infrastructure with clearly-documented motivation for each file. I verified via grep that no remaining file in the repo references any of the deleted paths, and confirmed ci.yml (lines 23-25) regenerates the default file on each run, so the committed copy is indeed non-load-bearing. The .github/ISSUE_TEMPLATE/ directory contains three form templates that shadow the deleted root issue_template.md.
Other factors
Recent commit history (85d7103, c76f860, 962fd8a) shows the deletions split into logical commits, each matching one cluster of removals described in the PR. No bugs were flagged by the bug hunting system. The PR is small, self-contained, and trivially reversible via git revert if any deletion turns out to be load-bearing.
2 participants
Description
One Line Summary
Removes unused Asana sync, Claude, and issue response-time workflows along with stale Travis config and superseded root files.
Details
Motivation
claude.yml,claude-code-review.yml) are redundant — Claude code review and@claudementions are provisioned separately by the platform team, so the per-repo YAMLs do nothing useful.set_response_time.ymlplus its two.jshelpers (os_probot_metadata.js,set_response_times.js) stamp aresponse_time_in_business_daysvalue into hidden probot metadata on each issue. Nothing reads the metric..travis.ymlpins Xcode 11.1 / iPhone 8 Plus / iOS 13.1 (2019); active CI is.github/workflows/ci.ymlon Xcode 16.4.issue_template.mdat the repo root is shadowed by.github/ISSUE_TEMPLATE/*.ymlform templates.defaultis overwritten byci.ymlon every run, so the committed copy isn't load-bearing.