mirror of
https://github.com/discourse/discourse.git
synced 2024-11-26 02:40:53 -06:00
30990006a9
This reduces chances of errors where consumers of strings mutate inputs and reduces memory usage of the app. Test suite passes now, but there may be some stuff left, so we will run a few sites on a branch prior to merging
74 lines
2.6 KiB
Ruby
74 lines
2.6 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
if github.pr_json && (github.pr_json["additions"] || 0) > 250 || (github.pr_json["deletions"] || 0) > 250
|
|
warn("This pull request is big! We prefer smaller PRs whenever possible, as they are easier to review. Can this be split into a few smaller PRs?")
|
|
end
|
|
|
|
prettier_offenses = `yarn --silent prettier --list-different "app/assets/stylesheets/**/*.scss" "app/assets/javascripts/**/*.es6" "test/javascripts/**/*.es6"`.split("\n")
|
|
|
|
unless prettier_offenses.empty?
|
|
fail(%{
|
|
This PR doesn't match our required code formatting standards, as enforced by prettier.io. <a href='https://meta.discourse.org/t/prettier-code-formatting-tool/93212'>Here's how to set up prettier in your code editor.</a>\n
|
|
#{prettier_offenses.map { |o| github.html_link(o) }.join("\n")}
|
|
})
|
|
end
|
|
|
|
locales_changes = git.modified_files.grep(/config\/locales/)
|
|
has_non_en_locales_changes = locales_changes.grep_v(/config\/locales\/(client|server)\.en\.yml/).any?
|
|
|
|
if locales_changes.any? && has_non_en_locales_changes
|
|
fail("Please submit your non-English translation updates via [Transifex](https://www.transifex.com/discourse/discourse-org/). You can read more on how to contribute translations [here](https://meta.discourse.org/t/contribute-a-translation-to-discourse/14882).")
|
|
end
|
|
|
|
files = (git.added_files + git.modified_files)
|
|
.select { |path| !path.start_with?("plugins/") }
|
|
.select { |path| path.end_with?("es6") || path.end_with?("rb") }
|
|
|
|
js_files = files.select { |path| path.end_with?(".js.es6") }
|
|
js_test_files = js_files.select { |path| path.end_with?("-test.js.es6") }
|
|
|
|
super_offenses = []
|
|
self_offenses = []
|
|
js_files.each do |path|
|
|
diff = git.diff_for_file(path)
|
|
|
|
next if !diff
|
|
|
|
diff.patch.lines.grep(/^\+\s\s/).each do |added_line|
|
|
super_offenses << path if added_line['this._super()']
|
|
self_offenses << path if added_line[/(?:(^|\W)self\.?)/]
|
|
end
|
|
end
|
|
|
|
jquery_find_offenses = []
|
|
js_test_files.each do |path|
|
|
diff = git.diff_for_file(path)
|
|
|
|
next if !diff
|
|
|
|
diff.patch.lines.grep(/^\+\s\s/).each do |added_line|
|
|
jquery_find_offenses << path if added_line['this.$(']
|
|
end
|
|
end
|
|
|
|
if !self_offenses.empty?
|
|
warn(%{
|
|
Use fat arrow instead of self pattern.\n
|
|
#{self_offenses.uniq.map { |o| github.html_link(o) }.join("\n")}
|
|
})
|
|
end
|
|
|
|
if !super_offenses.empty?
|
|
warn(%{
|
|
When possible use `this._super(...arguments)` instead of `this._super()`\n
|
|
#{super_offenses.uniq.map { |o| github.html_link(o) }.join("\n")}
|
|
})
|
|
end
|
|
|
|
if !jquery_find_offenses.empty?
|
|
warn(%{
|
|
Use `find()` instead of `this.$` in js tests`\n
|
|
#{jquery_find_offenses.uniq.map { |o| github.html_link(o) }.join("\n")}
|
|
})
|
|
end
|