Automate All the Things

Slide 1

Slide 1 text

AUTOMATE ALL THE THINGS 1

Slide 2

Slide 2 text

͋ΒΏΔ࡞ۀΛࣗಈԽ͠Α͏ 2

Slide 3

Slide 3 text

SELF INTRO ▸ Joshua Kaplan ▸ minne @ GMO Pepabo ▸ Danger-Swift contributer ▸ Likes 3

Slide 4

Slide 4 text

CI IS THE ULTIMATE AUTOMATION TOOL 4

Slide 5

Slide 5 text

WHY? 5

Slide 6

Slide 6 text

▸ Fewer mistakes ▸ Faster ▸ More efficient ▸ More exciting 6

Slide 7

Slide 7 text

KEY CI TOOLS ▸ Fastlane ▸ Danger ▸ SwiftLint 7

Slide 8

Slide 8 text

NORMAL CI FLOW (LEVEL 1) 8

Slide 9

Slide 9 text

RUN TESTS FOR EACH PR 9

Slide 10

Slide 10 text

Slide 11

Slide 11 text

INTERMEDIATE CI FLOW (LEVEL 2) 11

Slide 12

Slide 12 text

LINTERS AND OTHER STATIC ANALYSIS 12

Slide 13

Slide 13 text

SWIFTLINT + DANGER swiftlint.lint_files inline_mode: true 13

Slide 14

Slide 14 text

Slide 15

Slide 15 text

CHECK STORYBOARDS 15

Slide 16

Slide 16 text

# storyboardʹͣΕ͍ͯΔϏϡʔ΍੍໿ͷෆඋ͕ͳ͍͔ parsed_diff.each do |changed_file| next unless changed_file.file.match(/.*\.(storyboard)$/) changed_file.changed_lines.each do |changed_line| content = changed_line.content has_misplaced_view = content.include? 'misplaced="YES"' has_ambiguous_view = content.include? 'ambiguous="YES"' file = changed_file.file line_number = changed_line.number if has_misplaced_view fail('ͣΕ͍ͯΔϏϡʔ͕͋ΔͷͰɺUpdate FramesΛ࣮ߦ͍ͯͩ͘͠͞', file: file, line: line_number) end if has_ambiguous_view fail('੍໿͕ෆे෼ͳՕॴ͕͋ΔͷͰɺ௚͍ͯͩ͘͠͞', file: file, line: line_number) end end end 16

Slide 17

Slide 17 text

SPELL CHECKING 17

Slide 18

Slide 18 text

Slide 19

Slide 19 text

# λΠϙΛݕ஌͢Δ added_and_modified_files = git.added_files + git.modified_files added_and_modified_files.each do |file_path| next unless file_path =~ /\.swift$/ stdout, status = Open3.capture2("npx", "cspell", file_path) next if status.success? stdout.split("\n").each do |line| next unless matches = /\w+\.swift:(\d+).*-\sUnknown\sword\s$(\w+)$/.match(line) line_number = matches[1].to_i word = matches[2] warning = "λΠϙ͔΋ʁ #{word}" warn(warning, file: file_path, line: line_number) end end 19

Slide 20

Slide 20 text

AUTOMATE PR TASKS 20

Slide 21

Slide 21 text

ASSIGN REVIEWERS 21

Slide 22

Slide 22 text

# ϨϏϡϫʔࢦఆʢίϝϯτࢦఆͷͨΊʹ͜ΕΛҰ൪্ʹॻ͘ඞཁ͋Γʣ reviewers = ["user1", "user2", "user3"].reject { |reviewer| reviewer == github.pr_author } repo_name = github.pr_json["head"]["repo"]["full_name"] pr_number = github.pr_json["number"] number_of_comments = github.api.issue_comments(repo_name, pr_number).size if number_of_comments.zero? reviewers = reviewers.sample(2) github.api.request_pull_request_review( repo_name, pr_number, {}, "reviewers": reviewers ) end 22

Slide 23

Slide 23 text

POST CI RESULTS 23

Slide 24

Slide 24 text

Slide 25

Slide 25 text

# Xcode Summary build_report_file = 'build_results.json' xcode_summary.ignored_files = 'Pods/**' xcode_summary.ignores_warnings = true xcode_summary.inline_mode = true xcode_summary.report build_report_file 25

Slide 26

Slide 26 text

POST CODE COVERAGE 26

Slide 27

Slide 27 text

Slide 28

Slide 28 text

xcov.report( scheme: 'minne', workspace: 'minne.xcworkspace', exclude_targets: 'TodayExtension.appex, NotificationServiceExtension.appex, MinneKit.framework', ) 28

Slide 29

Slide 29 text

ADVANCED CI FLOW (LEVEL 3) 29

Slide 30

Slide 30 text

CUSTOM SWIFTLINT RULES 30

Slide 31

Slide 31 text

prefer_gregorian_calendar: name: "Gregorian Calendar" regex: "Calendar\\.current" message: "Please use `Calendar(identifier: .gregorian)` to avoid Japanese calendar-related bugs" severity: error https_only: name: "HTTPS Only" match_kinds: - string # ίϝϯτͳͲΛແࢹͯ͠ɺจࣈྻͷΈΛΈΔ regex: "http:" message: "Please use HTTPS due to security policy" severity: error 31

Slide 32

Slide 32 text

DELIVER BETA AND RELEASE BUILDS 32

Slide 33

Slide 33 text

lane :release do capture_screenshots # generate new screenshots for the App Store sync_code_signing(type: "appstore") # see code signing guide for more information build_app(scheme: "MyApp") upload_to_app_store # upload your app to App Store Connect slack(message: "Successfully uploaded a new App Store build") end 33

Slide 34

Slide 34 text

AUTOMATE SCREENSHOT DELIVERY 34

Slide 35

Slide 35 text

// App let app = XCUIApplication() setupSnapshot(app) app.launch() # Fastlane lane :screenshots do capture_screenshots frame_screenshots(white: true) upload_to_app_store end 35

Slide 36

Slide 36 text

CHECK FOR DEAD CODE 36

Slide 37

Slide 37 text

Slide 38

Slide 38 text

def get_dead_objc_code std_out, status = Open3.capture2("bundle", "exec", "fui", "--ignorexib", "--path", "minne/Classes") # fuiͷexitstatus͸ɺ࢖ΘΕ͍ͯͳ͍Ϋϥεͷ਺Ͱɺ # Bridging-Header͕ඞͣ౰ͨͬͯ͠·͏ͷͰɺ # 2ͭҎ্ͷ݁Ռ͕͋Δ৔߹௨஌͍ͯ͠Δɻ return status.exitstatus >= 2 ? std_out : "" end def get_dead_swift_code std_out, _ = Open3.capture2("periphery", "scan") r = /minne.*(Struct|Class) .*is unused$/ filtered_results_array = std_out.to_enum(:scan,r).map {$&}.flatten return filtered_results_array.join("\n") end # Main dead_objc_result = get_dead_objc_code dead_swift_result = get_dead_swift_code mention = "" message = "#{mention}, Unused files exist:\n" notify_slack(message + dead_objc_result) unless dead_objc_result == "" notify_slack(message + dead_swift_result) unless dead_swift_result == "" 38