Skip to content

Commit ecb311c

Browse files
committed
change cron interval min value interval instead of removing rule
1 parent 2e9eca6 commit ecb311c

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

WordPress-VIP-Go/ruleset.xml

+6-4
Original file line numberDiff line numberDiff line change
@@ -232,6 +232,12 @@
232232
<rule ref="WordPressVIPMinimum.Performance.WPQueryParams.PostNotIn">
233233
<severity>3</severity>
234234
</rule>
235+
<rule ref="WordPress.WP.CronInterval">
236+
<!-- cron is async on VIP batch containers, high frequency schedules will not impact performance. 60s is cron runner polling rate, so 60s is lowest on VIP -->
237+
<properties>
238+
<property name="min_interval" value="60"/>
239+
</properties>
240+
</rule>
235241

236242
<!-- Silence is golden, these don't affect us on VIP Go -->
237243
<rule ref="VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable">
@@ -251,9 +257,5 @@
251257
<rule ref="WordPress.WP.AlternativeFunctions.file_system_operations_fclose">
252258
<severity>0</severity>
253259
</rule>
254-
<rule ref="WordPress.WP.CronInterval.CronSchedulesInterval">
255-
<!-- VIP's cron infrastructure can handle frequent and heavy crons, lower schedule intervals will not slow down the performance of the site -->
256-
<severity>0</severity>
257-
</rule>
258260

259261
</ruleset>

0 commit comments

Comments
 (0)