Skip to content

ci: rename workflow ci→check on xcodeproject-v2 to match master#337

Open
Badlazzor wants to merge 4 commits into
xcodeproject-v2from
fix/bitrise-ci-workflow-rename
Open

ci: rename workflow ci→check on xcodeproject-v2 to match master#337
Badlazzor wants to merge 4 commits into
xcodeproject-v2from
fix/bitrise-ci-workflow-rename

Conversation

@Badlazzor

Copy link
Copy Markdown
Contributor

Summary

The unified step CI app (48fa8fbee698622c) invokes bitrise run check on every PR. The xcodeproject-v2 branch still has the older ci name, so every PR targeting it dies at workflow lookup:

```
specified Workflow (check) does not exist
```

Changes

  • bitrise.yml: rename cicheck (matches master).
  • Add trailing change-workdir: .. step in integration_test (also matches master — keeps the v2 branch in lock-step with mainline CI config).

Context

Spotted while triaging #335 under STEP-2426. Every open PR off xcodeproject-v2 is currently red on infra, not code.

Test plan

  • CI on this PR runs check successfully.

🤖 Generated with Claude Code

…nge-workdir

The unified step CI app (48fa8fbee698622c) invokes `bitrise run check`
on every PR. xcodeproject-v2 still had the older `ci` name, so every PR
targeting this branch failed at workflow lookup with
"specified Workflow (check) does not exist".

Bring bitrise.yml in line with master: rename `ci` → `check` and add
the trailing `change-workdir: ..` step that integration_test uses on
master.

Unblocks #335 (and any other open PR off this branch).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Badlazzor and others added 3 commits July 9, 2026 12:30
Fix pre-existing revive/staticcheck/gosimple issues surfaced when the CI workflow rename exposed them:
- add doc comments on exported XcodeProj + methods and workspace.NewFromFile
- io/ioutil.WriteFile → os.WriteFile (plist.go, recreate_schemes.go, xcodeproj.go)
- for true {} → for {} + drop unreachable trailing return in resolve()
- unused const invalid → blank identifier (preserve iota offset)
- schemes_test.go: assert err from Schemes()
- codesign tests: t.Errorf/Fatalf(err.Error()) → t.Error/Fatal(err)
- example_test.go: swallow projects with _ to satisfy SA4010
- resources_build_phase.go: explicit _ for optional entryPath

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.

2 participants