Skip to content

docs: add agents/roadmap; enable CI tests; add test matrix#1296

Open
takemiyamakoto wants to merge 33 commits intodevelopfrom
clankcompatible
Open

docs: add agents/roadmap; enable CI tests; add test matrix#1296
takemiyamakoto wants to merge 33 commits intodevelopfrom
clankcompatible

Conversation

@takemiyamakoto
Copy link

@takemiyamakoto takemiyamakoto commented Sep 17, 2025

  • Add AGENTS.md aligned to Android repo (iOS specifics)

  • Add ROADMAP.md with SDK alignment steps

  • Enable tests in Jenkins (appTests=true)

  • Add scripts/test-matrix.sh (Debug+Release)

  • Update README with testing instructions

@WRRicht3r
docs: add agents/roadmap; enable CI tests; add test matrix
d932f49 
- Add AGENTS.md aligned to Android repo (iOS specifics)

- Add ROADMAP.md with SDK alignment steps

- Enable tests in Jenkins (appTests=true)

- Add scripts/test-matrix.sh (Debug+Release)

- Update README with testing instructions
@takemiyamakoto
Copy link
Author

takemiyamakoto commented Sep 17, 2025

@codex

@chatgpt-codex-connector
Copy link

chatgpt-codex-connector bot commented Sep 17, 2025

Summary

  • Reviewed the new AGENTS.md, which now documents repository structure, build/test commands, coding standards, and agent expectations for the iOS project. AGENTS.mdL1-L95
  • Confirmed ROADMAP.md lays out the Polkadot SDK alignment plan, verification steps, and backlog items tailored for the iOS wallet. ROADMAP.mdL1-L69
  • Verified Jenkins now enables appTests so CI executes the iOS test suite by default. JenkinsfileL12-L26
  • Noted the README testing section now points contributors to the new matrix script for running both configurations locally. README.mdL11-L28
  • Checked scripts/test-matrix.sh, which covers Debug and Release runs, toggles ENABLE_TESTABILITY for Release, and gracefully handles missing xcpretty by falling back to raw xcodebuild output. scripts/test-matrix.shL1-L69

Testing

View task →

@WRRicht3r
ci: pin Xcode on Jenkins; fix FearlessKeys text resource; revert Code…
c001a69 
…cov workflow

Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r
ci: make native gem builds work on Jenkins; pin Xcode; fix FearlessKe…
2f6e4d0 
…ys text resource; revert Codecov workflow
@WRRicht3r
ci(jenkins): export SDKROOT/headers and toolchain for native gem builds
35bac31 
Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r
ci(jenkins): export SDKROOT/headers and toolchain for native gem builds
0a8f59c 
Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r
jenkins patch
6d6f546 
Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r
deps: bump FearlessKeys to 0.1.5; ci: scoped GitHub auth for private …
0ba5ee4 
…pods in Codecov

Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r
ci: fix Bundler native build on Jenkins; bump FearlessKeys to 0.1.5; …
bc400c5 
…scoped auth for private pods in Codecov
@WRRicht3r
ci(jenkins): escape in BUNDLE_BUILD__JSON to avoid Groovy interpolati…
7ab1c46 
…on, ci: make private pod auth optional; bump FearlessKeys to 0.1.5
@WRRicht3r
ci: roll back Codecov workflow and Jenkinsfile to last known working …
5af2850 
…versions
@soramitsu soramitsu deleted a comment from chatgpt-codex-connector bot Sep 19, 2025
@WRRicht3r
ci: revert to last known working configs; remove failing Podfile post…
bdf1d87 
…_install; reset FearlessKeys to 0.1.4
@WRRicht3r
ci: restore Jenkins appTests=false; revert Podfile to stable; keep Co…
094a10f 
…decov at last known config

Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r
ci(codecov): add optional scoped auth for private pods (GH_READ_TOKEN)
fe717ed 
Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r
ci: pin Xcode on Jenkins to 15.4; exclude FearlessKeys text payload f…
deca084 
…rom compile sources

Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r
ci: conditionally pin Xcode on Jenkins; exclude FearlessKeys text pay…
c23ac4f 
…load from compile sources

Signed-off-by: William Richter <richter@soramitsu.co.jp>
WRRicht3r
WRRicht3r previously approved these changes Sep 19, 2025
View reviewed changes
Cre-eD
Cre-eD previously approved these changes Sep 23, 2025
View reviewed changes
@WRRicht3r
ci(jenkins): auto-pin DEVELOPER_DIR to Xcode 15.x if present; keep Fe…
99a6d68 
…arlessKeys text exclusion

Signed-off-by: William Richter <richter@soramitsu.co.jp>
@WRRicht3r WRRicht3r dismissed stale reviews from Cre-eD and themself via 99a6d68 September 23, 2025 14:43
@WRRicht3r
fix(spm): switch shared-features-spm to main to fix IrohaCrypto modul…
35db258 
…emap error
@WRRicht3r
fix(spm): point shared-features-spm to master to resolve IrohaCrypto …
d849fbd 
…modulemap error
@WRRicht3r
chore(spm): pin shared-features-spm to last known good revision (2b13…
1c05abc 
…aea)
@WRRicht3r
ci/spm: pin shared-features-spm to 2b13aea and stub IrohaCrypto umbre…
fd0df5f 
…lla header during CI to fix modulemap on Xcode 18.x
@WRRicht3r
ci: run SPM resolve + IrohaCrypto umbrella stub inside node('mac-fear…
afe652d 
…less'); pin shared-features-spm to 2b13aea
@WRRicht3r
ci: patch IrohaCrypto module.modulemap and add umbrella header in inc…
e586a29 
…lude; pin shared-features-spm to 2b13aea
@WRRicht3r
ci: robustly patch IrohaCrypto module.modulemap and add umbrella head…
73b37e8 
…er in include; pin shared-features-spm to 2b13aea
@WRRicht3r
ci: fix sed quoting; patch IrohaCrypto modulemap and add umbrella hea…
8a2eeee 
…der; pin shared-features-spm
@WRRicht3r
ci: ensure IrohaCrypto umbrella header exists in both include/ and pa…
80cdad3 
…rent; keep modulemap patch; pin shared-features-spm
@WRRicht3r
ci(spm): clean SourcePackages caches before resolve; keep IrohaCrypto…
770dec2 
… umbrella/header patches
@WRRicht3r
fix(ci): unblock builds by bumping shared-features-spm and adding Iro…
fda7a51 
…haCrypto hotfix
@WRRicht3r
fix(spm): dedupe Web3 by switching to soramitsu/web3-swift (7.7.7)
927ac20
@WRRicht3r
fix(signing): use manual signing for app target to avoid Xcode accoun…
66daa19 
…t token requirement during archive
@WRRicht3r
fix(signing): set manual signing, identity and ad-hoc profile for Rel…
30af007 
…ease archive
@WRRicht3r
fix(signing): set Dev config to manual distribution with adhoc profile
cbf8e57
@WRRicht3r
fix(ci): make Google keys injection optional and non-fatal for PR builds
a587380
@WRRicht3r
chore(pods): gate FearlessKeys behind INCLUDE_FEARLESS_KEYS and limit…
e74ece5 
… to Release
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Cre-eD Cre-eD Cre-eD left review comments

@WRRicht3r WRRicht3r WRRicht3r left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@takemiyamakoto @Cre-eD @WRRicht3r