Skip to content

[fix] Replace "load" with "require_relative"#81

Open
rtfm98 wants to merge 1 commit into
paviliondev:mainfrom
rtfm98:fix_use_require_relative
Open

[fix] Replace "load" with "require_relative"#81
rtfm98 wants to merge 1 commit into
paviliondev:mainfrom
rtfm98:fix_use_require_relative

Conversation

@rtfm98

@rtfm98 rtfm98 commented Jul 14, 2026

Copy link
Copy Markdown

Replace "load" with "require_relative" as proposed by Discourse Dev Environment

Warning message

[{
	"resource": "/workspace/discourse/plugins/discourse-ratings/plugin.rb",
	"owner": "_generated_diagnostic_collection_name_#3",
	"code": "Discourse/Plugins/UseRequireRelative",
	"severity": 2,
	"message": "Discourse/Plugins/UseRequireRelative: Use `require_relative` instead of `load`.\n\nThis offense is not auto-correctable.\n",
	"source": "RuboCop",
	"startLineNumber": 90,
	"startColumn": 19,
	"endLineNumber": 90,
	"endColumn": 56,
	"origin": "extHost2"
}]

Change

Replaced load with require_relative

Tests

Tested in Discourse Dev Environment without problems

Replace "load" with "require_relative" as proposed by Discourse Dev Environment
@rtfm98 rtfm98 changed the title Update plugin.rb [fix] Replace "load" with "require_relative" Jul 14, 2026
@rtfm98
rtfm98 marked this pull request as ready for review July 14, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant