Dependabot config

[soutaro]

Follow up #1833.

• Limit open PR count to 3
• Skip auto-merge if .gemspec is updated, because it's a change to next release
• Skip auto-merge if Gemfile is updated, just to make it more conservative

[soutaro]

@ParadoxV5 Any thought?

[soutaro]

This doesn't work correctly yet...

[ParadoxV5]

This doesn't work correctly yet...

Good intentions, at least 👍

[ParadoxV5]

Slow Mode 😄

[ParadoxV5]

• Skip auto-merge if .gemspec is updated, because it's a change to next release
• Skip auto-merge if Gemfile is updated, just to make it more conservative

Looks like the Gemfile is also part of a release (only Gemfile.lock isn’t):

rbs/rbs.gemspec

Lines 28 to 34 in 6499a09

	# Specify which files should be added to the gem when it is released.
	# The `git ls-files -z` loads the files in the RubyGem that have been added into git.
	spec.files = Dir.chdir(File.expand_path('..', __FILE__)) do
	`git ls-files -z`.split("\x0").reject do |f|
	f.match(%r{^(test|spec|features|bin|steep|benchmark)/}) || f.match(/Gemfile\.lock/)
	end
	end

Although… Now I’m not sure if including Gemfile has a purpose?

[ParadoxV5]

Does this work?

Suggested change

	- '!Gemfile'
	- '!steep/Gemfile'
	- '!**/Gemfile'

[soutaro]

The */Gemfile pattern would work, but the ! doesn't work.

[ParadoxV5]

Here: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore

[soutaro]

Yeah, but the action didn't run with this PR. I expected to run it because this PR doesn't change none of the files in the ! pattern. So, I guess the negative (!) pattern doesn't work without positive pattern.

[soutaro]

Adding **/* at the top of patterns?

[soutaro]

Ah the if: section at the job section blocked running the job.

[soutaro]

Although… Now I’m not sure if including Gemfile has a purpose?

Right. Will remove the Gemfile from the .gem package too.

[ParadoxV5]

Or rather…?

Suggested change

	- '!rbs.gemspec'
	- '!Gemfile'
	- '!steep/Gemfile'
	- '**/Gemfile.lock'

[ParadoxV5]

BTW, leverage the path config you just learned so the CI don’t need to test everything for anything.

[soutaro]

Found that simply aborting the actions is sufficient. Give it a try!