version: 2

save_bundle_cache_option: &save_bundle_cache_option
  key: v2-bundle-{{ checksum "Gemfile.lock" }}
  paths:
    - ~/app/vendor/bundle

restore_bundle_cache_option: &restore_bundle_cache_option
  keys:
    - v2-bundle-{{ checksum "Gemfile.lock" }}
    - v2-bundle

jobs:
  rspec:
    <<: *default

    steps:
      - checkout
      - restore_cache:
          <<: *restore_bundle_cache_option
      - run: ./.circleci/setup_bundle.sh
      - save_cache:
          <<: *save_bundle_cache_option
      - run: ./.circleci/setup_database.sh

      - run: bundle exec rspec
      - run: bundle exec codeclimate-test-reporter

https://github.com/sue445/chatwork_mention_task/blob/bd6d7ec65d4bc0a8e38d55e082e740539737e538/.circleci/config.yml

rspec以外にrubocopとかの実行でもbundle installが必要なのでDRYにするためにrestore_cacheやsave_cache のオプションを抽出したり、bundle installの処理をsetup_bundle.sh に抽出しています。

#!/bin/bash -xe

bundle install --jobs=4 --retry=3 --path vendor/bundle
bundle clean

# Resolve bundler version difference between Gemfile.lock and pre-installed in CI
gem install restore_bundled_with --no-document
restore-bundled-with

https://github.com/sue445/chatwork_mention_task/blob/bd6d7ec65d4bc0a8e38d55e082e740539737e538/.circleci/setup_bundle.sh

After (CircleCI 2.1 preview)

restore_cache , bundle install, save_cache の組み合わせを setup_bundle というcommandとして抽出できたので再利用しやすくなりました。

version: 2.1

commands:
  setup_bundle:
    description: "Setup bundle with cache"
    steps:
      - restore_cache:
          keys:
          - v2-bundle-{{ checksum "Gemfile.lock" }}
          - v2-bundle
      - run:
          name: Setup bundle
          command: |
            set -xe
            bundle install --jobs=4 --retry=3 --path vendor/bundle
            bundle clean

            # Resolve bundler version difference between Gemfile.lock and pre-installed in CI
            gem install restore_bundled_with --no-document
            restore-bundled-with

      - save_cache:
          key: v2-bundle-{{ checksum "Gemfile.lock" }}
          paths:
          - ~/app/vendor/bundle

jobs:
  rspec:
    <<: *default

    steps:
      - checkout
      - setup_bundle
      - run: ./.circleci/setup_database.sh

      - run: bundle exec rspec
      - run: bundle exec codeclimate-test-reporter

https://github.com/sue445/chatwork_mention_task/blob/654e5137002bd611f44674d2e69b8ae1b4d31102/.circleci/config.yml

bundle install系の処理が1つになったことでBeforeにあったsave_bundle_cache_option や setup_bundle.sh にような工夫が不要になったのが嬉しい

所感

今回はcommandsしか使ってないですが他の機能を利用することでリファクタリングの余地が増えそう。

まだpreviewで色々変更ありそうなので個人プロダクト全部には投入しづらいけど、正式版になったら上のような修正を全部に入れたい。

作業PR

https://github.com/sue445/chatwork_mention_task/pull/184/files

2018-09-24

apple_system_status v1.0.0を出した

gem

自分以外使ってなさそうだけど一応アナウンス

github.com

大きな変更点

このgemについては過去のエントリを参照

sue445.hatenablog.com

大きな変更点

https://github.com/sue445/apple_system_status/blob/master/CHANGELOG.md#v100-2018-09-22 からピックアップ

phantomjsに依存してたのでHeadless Chromeを使うようにした

今更ですがHeadless Chromeに移行。

実は結構前に実装自体は終わっててローカルだとサクッと動いたのですが、CI周りで苦戦してました

簡単に書くと

最初TravisCIでやろうかと思ったらうまく動かない
- Chromeにつながらない
Docker Hubの circleci/ruby のbrowsersタグがついてるイメージならHeadless Chromeがインストール済だから大丈夫だろうと思ったらうまく動かない
- ChromeにはつながるんだけどDOM取得がうまくいかない
3ヶ月くらい経ってふとnode-browsersタグを使ったらCI通った

って感じ

https://github.com/sue445/apple_system_status/pull/25