Lint code

[rodrigoargumedo]

Hi,

I saw that the linting Actions job was failing and decided to correct it to pass.

From standardrb:

Style/SuperArguments: Call super without arguments and parentheses when the signature is identical

Please let me know if you have any questions for me.

[jcupitt]

Hi @rodrigoargumedo,

Ahhh they keep adding new rules and breaking our CI. Sure, happy to change this, though the lint is still failing (not sure why).

[rodrigoargumedo]

@jcupitt Thanks for the quick response. Do we want to enforce this rule or exclude it from being linted?

[jcupitt]

The rule sounds reasonable, so let's make this change. Thanks for volunteering!

[kleisauke]

It seems that the failing CI job was run on commit 7a2b9ba. I tried restarting it, but to no avail.
https://github.com/libvips/ruby-vips/actions/runs/10216871731/job/28270880174#step:2:70

[rodrigoargumedo]

It's failing due to branch is not equal to master and is on a feature branch that I forked from. So, this is a normal expected behavior to fail this way.

[rodrigoargumedo]

Once merged, it will pass.

[kleisauke]

Ah, I think it's due to this:

ruby-vips/.github/workflows/test.yml

Lines 8 to 10 in 7a2b9ba

	pull_request_target:
	branches:
	- master

[jcupitt]

Ah! Of course. I assumed it was linting the branch. Thanks again.

[kleisauke]

PR #403 removes this confusing pull_request_target event.