Skip to content

Remove unused `attr_reader` in triggerable_hooks.rb

Manoj M J requested to merge manojmj-master-patch-37603 into master

What does this MR do?

This MR removes an unused attr_reader in triggerable_hooks.rb

I noticed this by chance, while working on the review of !49285 (merged)

The introduction of this attr_reader was here and it appears to be un-used right from its introduction.

No changelog added as there is no user facing change.

Screenshots (strongly suggested)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Manoj M J

Merge request reports

Loading