SRGSSR / pillarbox-apple

A next-generation reactive media playback ecosystem for Apple platforms.
https://testflight.apple.com/join/TS6ngLqf
MIT License
54 stars 8 forks source link

Setup CI #3

Closed defagos closed 2 years ago

defagos commented 2 years ago
defagos commented 2 years ago

Danger can be used to report UT failure directly in GitHub comments. Some useful resources:

Since we are in a self-hosted CI scenario we need to:

See CI setup for the variables to provide.

defagos commented 2 years ago

I didn't clean the previous reports but pointed at the correct JUnit output instead.

defagos commented 2 years ago

To document for Danger integration:

defagos commented 2 years ago

Required on TeamCity before running Danger:

export GITHUB_REPO_SLUG=SRGSSR/pillarbox-apple
export GITHUB_REPO_URL=https://github.com/SRGSSR/pillarbox-apple
export GITHUB_PULL_REQUEST_ID=4

PR must be dynamic, of course.

The PR might be retrieved directly with teamcity.pullRequest.number but this might require special care when configuring the pull request plugin.

defagos commented 2 years ago

Jazzy is a bit too old to be integrated. Skipped.

defagos commented 2 years ago

I could not make spellchecking with proselint and mdspell work because of version conflicts. Skipped.

defagos commented 2 years ago

Using bundle install --path vendor/bundle is deprecated but the suggested replacement, namely setting bundle config set --local path 'vendor/bundle' first, leads to issues with fastlane:

bundler: failed to load command: fastlane (/Users/defagos/Developer/repositories/pillarbox-apple/vendor/bundle/ruby/3.1.0/bin/fastlane)
/Users/defagos/Developer/repositories/pillarbox-apple/vendor/bundle/ruby/3.1.0/gems/fastlane-2.208.0/fastlane/lib/fastlane/junit_generator.rb:13:in `read': No such file or directory @ rb_sysopen - /Users/defagos/Developer/repositories/pillarbox-apple/vendor/bundle/ruby/3.1.0/gems/fastlane-2.208.0/fastlane/lib/assets/report_template.xml.erb (Errno::ENOENT)

So for the moment I used neither.

defagos commented 2 years ago

Won't add a command to register testers, not so cumbersome and can be done with pilot if later needed.