Automated Bundler caching

[eregon]

I'd like to add automated Bundler caching in this action, since caching manually like https://github.com/ruby/setup-ruby/blob/master/README.md#caching-bundle-install is quite verbose and hard to get right (e.g., sass/sassc-ruby#183 (comment)).

It would be optional, not sure about the default yet.

For that to work, we need actions/cache to be a package, so this needs actions/cache#55 first.

[eregon]

WIP PR: #56

https://github.com/actions/toolkit/tree/master/packages/cache

[eregon]

#56 is merged, currently it's opt-in:

    - uses: ruby/setup-ruby@v1
      with:
        bundler-cache: true

[DannyBen]

Is this already deployed and should work?

With this step:

    - name: Setup Ruby
      uses: actions/setup-ruby@v1
      with: 
        ruby-version: '${{ matrix.ruby }}'
        bundler-cache: true

I am getting an error:

##[warning]Unexpected input(s) 'bundler-cache', valid inputs are ['ruby-version', 'version']

[eregon]

@DannyBen yes it's implemented and released. It's ruby/setup-ruby, not actions/setup-ruby.

[DannyBen]

Oh... I thought it was the official actions/setup-ruby.

Got a little confused there - and I see just as I am typing, that an issue "Should we deprecate this action" was linked from actions/setup-ruby...

[eregon]

@DannyBen To clarify, I'm proposing to deprecate actions/setup-ruby.
This action, ruby/setup-ruby has many advantages as you can see on that issue.
It's not official as in "made by GitHub" but it's "hosted in the official Ruby organization", and ruby/setup-ruby is the Ruby starter workflow (can be seen when creating a new action with the UI).

[DannyBen]

To clarify, I'm proposing to deprecate actions/setup-ruby.

Yes, I totally understand it, and although the ruby/* action is not the "official by GitHub", it is still "official by Ruby" - which is good (similar to AWS actions that are developed by Amazon team).

I expressed my opinion in that other ticket as well, thank you for clarifying.

... and the automatic cache is SUPER nice, makes the workflow file resemble the short and sweet travis YAML files. Thanks for this implementaiton.

[scottjacobsen]

The github provided https://github.com/actions/cache had performance problems until the more recent releases. Does bundler caching use the newest version of that action? I searched the code a bit but I can't really tell.

[dentarg]

@scottjacobsen I think we can understand what versions are used from yarn.lock:

setup-ruby/yarn.lock

Line 5 in bdfdb53

"@actions/cache@^0.2.1":

setup-ruby/yarn.lock

Line 55 in bdfdb53

"@actions/tool-cache@^1.3.1":

From https://www.npmjs.com/package/@actions/cache, 1.0.2 is the latest, and from https://www.npmjs.com/package/@actions/tool-cache, 1.6.0 is the latest right now.

So it does not look like setup-ruby is using the latest versions. If that should be acted on, should perhaps be discussed in a new issue/PR?

(EDIT: @actions/tool-cache is not related to the sort of caching discussed here, my bad)