f227f66b11
DEV: Trigger an event after updating topic hot scores ( #29261 )
2024-10-18 11:08:48 -03:00
e95edd079b
DEV: Refactor some core services
...
Extracted from https://github.com/discourse/discourse/pull/29129 .
This patch makes the code more compliant with the upcoming service docs
best practices.
2024-10-18 16:06:58 +02:00
007a8dabf3
FIX: create-invite-uploader following uppy-upload mixin changes ( #29278 )
...
This component will soon be updated to remove the mixin entirely (and add a regression test for it). But for now, this is a quick fix to get it working again.
2024-10-18 12:44:13 +01:00
16acba6cf8
DEV: serialize image upload thumbnail ( #29276 )
...
Since we recently blocked accidental serialization of AR models, we are getting a 500 error in some cases with thumbnails. We can fix this by serializing the thumbnail, previously we just returned a raw OptimizedImage object.
Thumbnails are now attached to the serializer in core, therefore we no longer need to use add_to_serializer within the chat plugin to use thumbnails within chat message uploads.
2024-10-18 12:55:14 +04:00
7f607699b8
DEV: Refactor flag related services a bit
...
Extracted from https://github.com/discourse/discourse/pull/29129 .
This patch makes the code more compliant with the upcoming service docs
best practices.
2024-10-18 10:10:28 +02:00
f8360f9665
FIX: Don't error out on nested topic show id param ( #29274 )
...
We're expecting the ID param to be something that neatly coerces into an ID. If we receive something like a nested parameter, this will blow up. (We already handle the case of arrays.)
This commit raises an InvalidParameters exception in the case of a nested ID.
2024-10-18 14:37:52 +08:00
9dafbe47dc
FIX: Don't error out on nested categories index page param ( #29273 )
...
We're expecting the page param to be something that neatly coerces into an integer. If we receive something like a nested parameter, this will blow up. (I'm sure there are other examples as well.)
This commit falls back to a page value of 1 if the coercion fails.
2024-10-18 14:37:39 +08:00
48308a5ee6
FIX: show lightbox for small images ( #29140 )
...
We want to allow lightboxing of smaller images, even if they are below the minimum size for image thumbnail generation.
This change sets a minimum threshold of 100 x 100 pixels for triggering the lightbox.
---------
Co-authored-by: Régis Hanol <regis@hanol.fr>
2024-10-18 09:45:08 +04:00
11017b20f8
UX: ensure ios font size is always min 16px with max() fn ( #29270 )
2024-10-18 05:26:12 +02:00
fdd4804a8d
FIX: do not overrider trigger when equals 0 ( #29269 )
...
This commit is fixing the path which sets a default value to trigger. We were doing `if (!this.model.trigger)` but `this.model.trigger` can have `0` as value, which would trigger this codepath and this codepath was setting the first value of `badgeTriggers` as a default value for trigger.
2024-10-18 08:34:34 +09:00
faa9489245
Build(deps): Bump net-imap from 0.4.17 to 0.5.0 ( #29267 )
...
Bumps [net-imap](https://github.com/ruby/net-imap ) from 0.4.17 to 0.5.0.
- [Release notes](https://github.com/ruby/net-imap/releases )
- [Commits](https://github.com/ruby/net-imap/compare/v0.4.17...v0.5.0 )
---
updated-dependencies:
- dependency-name: net-imap
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2024-10-18 00:27:04 +02:00
ba417728d0
Build(deps-dev): Bump lefthook from 1.7.18 to 1.7.21 ( #29265 )
...
Bumps [lefthook](https://github.com/evilmartians/lefthook ) from 1.7.18 to 1.7.21.
- [Release notes](https://github.com/evilmartians/lefthook/releases )
- [Changelog](https://github.com/evilmartians/lefthook/blob/master/CHANGELOG.md )
- [Commits](https://github.com/evilmartians/lefthook/compare/v1.7.18...v1.7.21 )
---
updated-dependencies:
- dependency-name: lefthook
dependency-type: direct:development
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2024-10-18 00:14:52 +02:00
7f15fdb611
DEV: Make the dev watcher ignore node_modules ( #29263 )
2024-10-17 23:34:10 +02:00
c1f078cacf
FIX: Make subcategories page more like categories ( #29240 )
...
The subcategories page was not paginated and it was using the
subcategory style from the category settings. The same page style should
be used for categories and subcategories page.
2024-10-17 19:47:53 +03:00
23c486799f
DEV: Improve `array` type in service contracts
...
This patch improves the custom `array` type available in contracts.
It’s now able to split strings on `|` on top of `,`, and to be more
consistent, it also tries to cast the resulting items to integers.
2024-10-17 17:02:02 +02:00
2ee63e4164
FEATURE: Enable image optimization on iOS >= 18 ( #29257 )
2024-10-17 11:51:50 -03:00
4bb9e0e8c9
DEV: Correct link in uppy-debugging warning message ( #29256 )
...
Followup to 06d32a8a
2024-10-17 15:30:18 +01:00
c08c40dc23
FEATURE: theme_modifiers can depend on theme settings (plus serialize_post_user_badges) ( #29227 )
...
Theme modifiers can now be defined as theme settings, this allows for
site operators to override behavior of theme modifiers.
New syntax is:
```
{
...
"modifiers": {
"modifier_name": {
"type": "setting",
"value": "setting_name"
}
}
}
```
This also introduces a new theme modifier for serialize_post_user_badges. Name of badge must match the name of the badge in the badges table. The client-side is updated to load this new data from the post-stream serializer.
Co-authored-by: David Taylor <david@taylorhq.com>
2024-10-17 15:16:16 +01:00
54935219d7
DEV: fix flakey spec in sidebar ( #29254 )
...
Technically we don't show the edit custom section button on mobile, but the button is present so I just fixed it so the finder works on mobile. We should probably remove this test or find a way to make the button visible on mobile.
Also used `mobile: true` instead of manual url.
2024-10-17 22:28:02 +09:00
af6788fd33
DEV: Extract leave logic to the `Chat::Channel` model
...
Extracted from https://github.com/discourse/discourse/pull/29129 .
2024-10-17 14:58:45 +02:00
f05b984208
DEV: glimmerify discourse-presence ( #29235 )
2024-10-17 14:51:26 +02:00
7b8ebebe93
DEV: Fix regression in uppy-mixin ( #29252 )
...
This API change was unintentionally introduced in 06d32a8a8923d90ecb26
2024-10-17 10:38:19 +01:00
93e02069b0
DEV: Don’t provide an array to site settings group lists in specs
...
This is extracted from https://github.com/discourse/discourse/pull/29129 .
In some chat specs, we provide an array as a value for group lists like
`chat_allowed_groups`, which is wrong. This results in a value like
`"1|2|[3]"` instead of `"1|2|3"`.
2024-10-17 11:25:31 +02:00
5d657c8c41
DEV: Remove flaky test for now ( #29249 )
...
Adding the directory item test causes the default test to fail randomly due to directory items not getting removed properly.
Removing this for now, and also moving this test to the common system folder instead of system/user_page
2024-10-17 16:58:35 +08:00
bfb5eaf21a
UX: fix sidebar styling on tablet ( #29248 )
2024-10-17 07:32:57 +02:00
34259f8735
FIX: Add back user custom field plugin outlet that was accidentally removed ( #29247 )
...
When converting the user custom fields admin form in #29070 , I accidentally removed the plugin outlet after-admin-user-fields. This is used by the discourse-authentication-validations plugin, which is now broken on main core.
This commit adds back the plugin outlet in core.
2024-10-17 13:06:55 +08:00
23d90ecb26
FIX: _uploadDropTargetOptions is now public ( #29246 )
...
Chat was using a private function which has now been made public.
This commit also adds a test for this codepath.
2024-10-17 13:10:01 +09:00
070afcf605
Build(deps-dev): Bump rubocop-discourse from 3.8.1 to 3.8.2 ( #29243 )
...
Bumps [rubocop-discourse](https://github.com/discourse/rubocop-discourse ) from 3.8.1 to 3.8.2.
- [Commits](https://github.com/discourse/rubocop-discourse/compare/v3.8.1...v3.8.2 )
---
updated-dependencies:
- dependency-name: rubocop-discourse
dependency-type: direct:development
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2024-10-17 00:16:45 +02:00
72687e31a1
Build(deps-dev): Bump puppeteer-core from 23.5.3 to 23.6.0 ( #29241 )
...
Bumps [puppeteer-core](https://github.com/puppeteer/puppeteer ) from 23.5.3 to 23.6.0.
- [Release notes](https://github.com/puppeteer/puppeteer/releases )
- [Changelog](https://github.com/puppeteer/puppeteer/blob/main/release-please-config.json )
- [Commits](https://github.com/puppeteer/puppeteer/compare/puppeteer-core-v23.5.3...puppeteer-core-v23.6.0 )
---
updated-dependencies:
- dependency-name: puppeteer-core
dependency-type: direct:development
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2024-10-17 00:03:19 +02:00
260907c37b
Build(deps): Bump terser from 5.34.1 to 5.36.0 ( #29242 )
...
Bumps [terser](https://github.com/terser/terser ) from 5.34.1 to 5.36.0.
- [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md )
- [Commits](https://github.com/terser/terser/compare/v5.34.1...v5.36.0 )
---
updated-dependencies:
- dependency-name: terser
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2024-10-16 23:41:58 +02:00
da571b6ac5
DEV: add before and after mobile categories outlet ( #29238 )
2024-10-16 16:30:19 -04:00
f28f82f99e
UX: Multiple fixes to theme card rendering ( #29225 )
...
* Improvements, refactors, remove dead code
---------
Co-authored-by: Martin Brennan <martin@discourse.org>
2024-10-16 11:13:36 -05:00
6e6dbde898
DEV: move chat time formats to core locales ( #29236 )
...
This change moves the date/time formats to core locales, so that they can be used outside of the plugin.
2024-10-16 20:10:40 +04:00
67b6997306
DEV: Add `SearchMenu::Results::Assistant` plugin outlets
...
Add plugin outlets for each assistant result type - User, Tag and Categories, Groups, etc. This gives us the ability to add content for each _type_ of search suggestion:
<img width="216" alt="Screenshot 2024-10-16 at 10 01 09 AM" src="https://github.com/user-attachments/assets/fbbc71fe-a8fe-499b-8377-480dd0ed5f75 ">
I would have preferred to add a single plugin outlet at the top of the template and pass `this.suggestionType` and `@results`, but that would then require that we carry over a ton of core logic to plugins to calculate which _type_ is being rendered, and it would get unnecessarily messy quick. So instead I opted to create a plugin outlet for each _type_.
2024-10-16 10:46:15 -05:00
76aa9b66d0
DEV: Allow `ChatSDK.create` to take extra parameters
...
Currently, `ChatSDK.create` restricts what parameters can be provided to
the underlying service. This prevents the `discourse-ai` plugin from
using it in one of its specs.
This patch allows extra parameters to be provided.
2024-10-16 17:34:01 +02:00
789aa2d9de
DEV: Add PluginRegistry modifiers to #review and #recalculate ( #29128 )
...
* DEV: Add PluginRegistry modifiers to #review and #recalculate
* added tests
* changed added registry logic
2024-10-16 10:26:10 -05:00
6078fb73ea
UX: consistent chat message date format ( #29232 )
...
In certain locales like English (GB), If a user posted 2 subsequent messages, the first would have a date displayed in 24 hour format, while the second message would be shown in 12 hour format (when hovering the message).
This change forces both messages to display in 12 hour format, the first message showing the am/pm, and the second showing the smaller version without am/pm.
2024-10-16 21:51:04 +08:00
4aa923aab1
FIX: Support add_directory_column in glimmered table header toggle ( #29231 )
...
#29209 introduced a bug where columns to the directory added via add_directory_column are not being translated properly.
This fixes the issue and adds an integration test.
2024-10-16 21:49:01 +08:00
8ee872f7e5
DEV: Modernize tags-uploader component ( #29215 )
2024-10-16 11:54:59 +01:00
7dc60d0c99
DEV: Remove discourse-encrypt from core CI suite ( #29230 )
...
This plugin is prone to causing flaky test runs, and is in the process of being deprecated
2024-10-16 11:47:02 +01:00
06d32a8a89
DEV: Refactor uppy component mixins into standalone classes ( #28710 )
...
This commit replaces all uppy-related mixins with standalone classes. The main entrypoint is now lib/uppy/uppy-upload.js, which has a list of its config options listed at the top of the file. Functionality & logic is completely unchanged.
The uppy-upload mixin is replaced with a backwards-compatibility shim, which will allow us to migrate to the new pattern incrementally.
2024-10-16 11:15:19 +01:00
6a7bac7694
Build(deps-dev): Bump @glint/core from 1.4.1-unstable.ff9ea6c to 1.5.0 ( #29219 )
...
Bumps [@glint/core](https://github.com/typed-ember/glint ) from 1.4.1-unstable.ff9ea6c to 1.5.0.
- [Release notes](https://github.com/typed-ember/glint/releases )
- [Changelog](https://github.com/typed-ember/glint/blob/main/V2_CHANGELOG.md )
- [Commits](https://github.com/typed-ember/glint/commits/1.5.0 )
---
updated-dependencies:
- dependency-name: "@glint/core"
dependency-type: direct:development
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2024-10-16 11:51:21 +02:00
78ed8ede8a
DEV: Improve isolation and concurrency for minio-based upload specs ( #29216 )
...
- Uses a temporary, clean, per-test-process directory for minio data
- Runs a separate minio instance for each test process
- Unskips minio-based tests in CI
2024-10-16 10:40:58 +01:00
e5f2416eff
Update translations ( #29212 )
2024-10-16 17:52:01 +09:00
305927fa4b
DEV: Refactor categories controller to reuse code ( #29172 )
2024-10-16 11:41:26 +03:00
c8a94b5b3d
FIX: Prevent focus shift when navigating the user directory ( #29209 )
...
Prevent focus shift when filtering in the user directory.
Also glimmerifies the table header toggle
2024-10-16 11:37:08 +08:00
35284c77f1
Build(deps-dev): Bump rubocop from 1.66.1 to 1.67.0 ( #29226 )
...
Bumps [rubocop](https://github.com/rubocop/rubocop ) from 1.66.1 to 1.67.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a href="https://github.com/rubocop/rubocop/releases ">rubocop's releases</a>.</em></p>
<blockquote>
<h2>RuboCop 1.67</h2>
<h3>New features</h3>
<ul>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13259 ">#13259</a>: Add new <code>Lint/DuplicateSetElement</code> cop. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13223 ">#13223</a>: Add <code>AllowRBSInlineAnnotation</code> config option to <code>Layout/LeadingCommentSpace</code> to support RBS::Inline style annotation comments. ([<a href="https://github.com/tk0miya "><code>@tk0miya</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13310 ">#13310</a>: Display analysis Ruby version in <code>rubocop -V</code>. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
</ul>
<h3>Bug fixes</h3>
<ul>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13314 ">#13314</a>: Fix a false negative for <code>Style/Semicolon</code> when using a semicolon between a closing parenthesis after a line break and a consequent expression. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13217 ">#13217</a>: Fix a false positive in <code>Lint/ParenthesesAsGroupedExpression</code> with compound ranges. ([<a href="https://github.com/gsamokovarov "><code>@gsamokovarov</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13268 ">#13268</a>: Fix a false positive for <code>Style/BlockDelimiters</code> when a single line do-end block with an inline <code>rescue</code> with a semicolon before <code>rescue</code>. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13298 ">#13298</a>: Fix an error for <code>Layout/AccessModifierIndentation</code> when the access modifier is on the same line as the class definition. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13198 ">#13198</a>: Fix an error for <code>Style/OneLineConditional</code> when using nested if/then/else/end. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13316 ">#13316</a>: Fix an incorrect autocorrect for <code>Lint/ImplicitStringConcatenation</code> with <code>Lint/TripleQuotes</code> when string literals with triple quotes are used. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13220 ">#13220</a>: Fix an incorrect autocorrect for <code>Style/ArgumentsForwarding</code> when using only forwarded arguments in brackets. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13202 ">#13202</a>: Fix an incorrect autocorrect for <code>Style/CombinableLoops</code> when looping over the same data with different block variable names. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13291 ">#13291</a>: Fix an incorrect autocorrect for <code>Style/RescueModifier</code> when using modifier rescue for method call with heredoc argument. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13226 ">#13226</a>: Fix <code>--auto-gen-config</code> when passing an absolute config path. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13225 ">#13225</a>: Avoid syntax error when correcting <code>Style/OperatorMethodCall</code> with <code>/</code> operations followed by a parenthesized argument. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13235 ">#13235</a>: Fix an error for <code>Style/IfUnlessModifier</code> when multiline <code>if</code> that fits on one line and using implicit method call with hash value omission syntax. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13219 ">#13219</a>: Fix a false positive for <code>Style/ArgumentsForwarding</code> with Ruby 3.0 and optional position arguments. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13271 ">#13271</a>: Fix a false positive for <code>Lint/AmbiguousRange</code> when using rational literals. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13260 ">#13260</a>: Fix a false positive for <code>Lint/RedundantSafeNavigation</code> with namespaced constants. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13224 ">#13224</a>: Fix false positives for <code>Style/OperatorMethodCall</code> with named forwarding. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13213 ">#13213</a>: Fix false positives for <code>Style/AccessModifierDeclarations</code> when <code>AllowModifiersOnAttrs: true</code> and using splat with a percent symbol array, or with a constant. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13145 ">#13145</a>: Fix false positives for <code>Style/RedundantLineContinuation</code> when line continuations with comparison operator and the LHS is wrapped in parentheses. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/12875 ">#12875</a>: Fix false positive for <code>Style/ArgumentsForwarding</code> when argument is used inside a block. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13239 ">#13239</a>: Fix false positive for <code>Style/CollectionCompact</code> when using <code>delete_if</code>. ([<a href="https://github.com/masato-bkn "><code>@masato-bkn</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13210 ">#13210</a>: Fix omit_parentheses style for pattern match with value omission in single-line branch. ([<a href="https://github.com/gsamokovarov "><code>@gsamokovarov</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13149 ">#13149</a>: Handle crashes in custom Ruby extractors more gracefully. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13319 ">#13319</a>: Handle literal forward slashes inside a <code>regexp</code> in <code>Lint/LiteralInInterpolation</code>. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13208 ">#13208</a>: Fix an incorrect autocorrect for <code>Style/IfWithSemicolon</code> when single-line <code>if/;/end</code> when the then body contains a method call with <code>[]</code> or <code>[]=</code>. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13318 ">#13318</a>: Prevent modifying blocks with <code>Style/HashEachMethods</code> if the hash is modified within the block. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13293 ">#13293</a>: Fix <code>TargetRubyVersion</code> from a gemspec when the gemspec is not named like the folder it is located in. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13211 ">#13211</a>: Fix wrong autocorrect for <code>Style/GuardClause</code> when using heredoc without <code>else</code> branch. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13215 ">#13215</a>: Fix wrong autocorrect for <code>Lint/BigDecimalNew</code> when using <code>::BigDecimal.new</code>. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13215 ">#13215</a>: Fix wrong autocorrect for <code>Style/MethodCallWithArgsParentheses</code> with <code>EnforcedStyle: omit_parentheses</code> and whitespace. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13302 ">#13302</a>: Fix incompatible autocorrect between <code>Style/RedundantBegin</code> and <code>Style/BlockDelimiters</code> with <code>EnforcedStyle: braces_for_chaining</code>. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
</ul>
<h3>Changes</h3>
<ul>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13221 ">#13221</a>: Do not group accessors having RBS::Inline annotation comments in <code>Style/AccessorGrouping</code>. ([<a href="https://github.com/tk0miya "><code>@tk0miya</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13286 ">#13286</a>: Add <code>AllowedMethods</code> configuration to <code>Layout/FirstMethodArgumentLineBreak</code>. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13110 ">#13110</a>: Add support in <code>Style/ArgumentsForwarding</code> for detecting forwarding of all anonymous arguments. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13222 ">#13222</a>: Allow to write RBS::Inline annotation comments after method definition in <code>Style/CommentedKeyword</code>. ([<a href="https://github.com/tk0miya "><code>@tk0miya</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13253 ">#13253</a>: Emit a deprecation when custom cops inherit from <code>RuboCop::Cop::Cop</code>. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13300 ">#13300</a>: Set <code>EnforcedShorthandSyntax: either</code> by default for <code>Style/HashSyntax</code>. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13254 ">#13254</a>: Enhance the autocorrect for <code>Naming/InclusiveLanguage</code> when a sole suggestion is set. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13232 ">#13232</a>: Make server mode aware of auto-restart for local config update. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
</ul>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a href="https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md ">rubocop's changelog</a>.</em></p>
<blockquote>
<h2>1.67.0 (2024-10-15)</h2>
<h3>New features</h3>
<ul>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13259 ">#13259</a>: Add new <code>Lint/DuplicateSetElement</code> cop. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13223 ">#13223</a>: Add <code>AllowRBSInlineAnnotation</code> config option to <code>Layout/LeadingCommentSpace</code> to support RBS::Inline style annotation comments. ([<a href="https://github.com/tk0miya "><code>@tk0miya</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13310 ">#13310</a>: Display analysis Ruby version in <code>rubocop -V</code>. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
</ul>
<h3>Bug fixes</h3>
<ul>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13314 ">#13314</a>: Fix a false negative for <code>Style/Semicolon</code> when using a semicolon between a closing parenthesis after a line break and a consequent expression. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13217 ">#13217</a>: Fix a false positive in <code>Lint/ParenthesesAsGroupedExpression</code> with compound ranges. ([<a href="https://github.com/gsamokovarov "><code>@gsamokovarov</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13268 ">#13268</a>: Fix a false positive for <code>Style/BlockDelimiters</code> when a single line do-end block with an inline <code>rescue</code> with a semicolon before <code>rescue</code>. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13298 ">#13298</a>: Fix an error for <code>Layout/AccessModifierIndentation</code> when the access modifier is on the same line as the class definition. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13198 ">#13198</a>: Fix an error for <code>Style/OneLineConditional</code> when using nested if/then/else/end. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13316 ">#13316</a>: Fix an incorrect autocorrect for <code>Lint/ImplicitStringConcatenation</code> with <code>Lint/TripleQuotes</code> when string literals with triple quotes are used. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13220 ">#13220</a>: Fix an incorrect autocorrect for <code>Style/ArgumentsForwarding</code> when using only forwarded arguments in brackets. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13202 ">#13202</a>: Fix an incorrect autocorrect for <code>Style/CombinableLoops</code> when looping over the same data with different block variable names. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13291 ">#13291</a>: Fix an incorrect autocorrect for <code>Style/RescueModifier</code> when using modifier rescue for method call with heredoc argument. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13226 ">#13226</a>: Fix <code>--auto-gen-config</code> when passing an absolute config path. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13225 ">#13225</a>: Avoid syntax error when correcting <code>Style/OperatorMethodCall</code> with <code>/</code> operations followed by a parenthesized argument. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13235 ">#13235</a>: Fix an error for <code>Style/IfUnlessModifier</code> when multiline <code>if</code> that fits on one line and using implicit method call with hash value omission syntax. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13219 ">#13219</a>: Fix a false positive for <code>Style/ArgumentsForwarding</code> with Ruby 3.0 and optional position arguments. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13271 ">#13271</a>: Fix a false positive for <code>Lint/AmbiguousRange</code> when using rational literals. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13260 ">#13260</a>: Fix a false positive for <code>Lint/RedundantSafeNavigation</code> with namespaced constants. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13224 ">#13224</a>: Fix false positives for <code>Style/OperatorMethodCall</code> with named forwarding. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13213 ">#13213</a>: Fix false positives for <code>Style/AccessModifierDeclarations</code> when <code>AllowModifiersOnAttrs: true</code> and using splat with a percent symbol array, or with a constant. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13145 ">#13145</a>: Fix false positives for <code>Style/RedundantLineContinuation</code> when line continuations with comparison operator and the LHS is wrapped in parentheses. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/12875 ">#12875</a>: Fix false positive for <code>Style/ArgumentsForwarding</code> when argument is used inside a block. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13239 ">#13239</a>: Fix false positive for <code>Style/CollectionCompact</code> when using <code>delete_if</code>. ([<a href="https://github.com/masato-bkn "><code>@masato-bkn</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13210 ">#13210</a>: Fix omit_parentheses style for pattern match with value omission in single-line branch. ([<a href="https://github.com/gsamokovarov "><code>@gsamokovarov</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13149 ">#13149</a>: Handle crashes in custom Ruby extractors more gracefully. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13319 ">#13319</a>: Handle literal forward slashes inside a <code>regexp</code> in <code>Lint/LiteralInInterpolation</code>. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13208 ">#13208</a>: Fix an incorrect autocorrect for <code>Style/IfWithSemicolon</code> when single-line <code>if/;/end</code> when the then body contains a method call with <code>[]</code> or <code>[]=</code>. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13318 ">#13318</a>: Prevent modifying blocks with <code>Style/HashEachMethods</code> if the hash is modified within the block. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13293 ">#13293</a>: Fix <code>TargetRubyVersion</code> from a gemspec when the gemspec is not named like the folder it is located in. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13211 ">#13211</a>: Fix wrong autocorrect for <code>Style/GuardClause</code> when using heredoc without <code>else</code> branch. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13215 ">#13215</a>: Fix wrong autocorrect for <code>Lint/BigDecimalNew</code> when using <code>::BigDecimal.new</code>. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13215 ">#13215</a>: Fix wrong autocorrect for <code>Style/MethodCallWithArgsParentheses</code> with <code>EnforcedStyle: omit_parentheses</code> and whitespace. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13302 ">#13302</a>: Fix incompatible autocorrect between <code>Style/RedundantBegin</code> and <code>Style/BlockDelimiters</code> with <code>EnforcedStyle: braces_for_chaining</code>. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
</ul>
<h3>Changes</h3>
<ul>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13221 ">#13221</a>: Do not group accessors having RBS::Inline annotation comments in <code>Style/AccessorGrouping</code>. ([<a href="https://github.com/tk0miya "><code>@tk0miya</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13286 ">#13286</a>: Add <code>AllowedMethods</code> configuration to <code>Layout/FirstMethodArgumentLineBreak</code>. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/issues/13110 ">#13110</a>: Add support in <code>Style/ArgumentsForwarding</code> for detecting forwarding of all anonymous arguments. ([<a href="https://github.com/dvandersluis "><code>@dvandersluis</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13222 ">#13222</a>: Allow to write RBS::Inline annotation comments after method definition in <code>Style/CommentedKeyword</code>. ([<a href="https://github.com/tk0miya "><code>@tk0miya</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13253 ">#13253</a>: Emit a deprecation when custom cops inherit from <code>RuboCop::Cop::Cop</code>. ([<a href="https://github.com/earlopain "><code>@earlopain</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13300 ">#13300</a>: Set <code>EnforcedShorthandSyntax: either</code> by default for <code>Style/HashSyntax</code>. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
<li><a href="https://redirect.github.com/rubocop/rubocop/pull/13254 ">#13254</a>: Enhance the autocorrect for <code>Naming/InclusiveLanguage</code> when a sole suggestion is set. ([<a href="https://github.com/koic "><code>@koic</code></a>][])</li>
</ul>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="c108ac082274ddd9bb96b5c12aea89c936160665https://redirect.github.com/rubocop/rubocop/issues/13331 ">#13331</a>] Fix an error when using release task</li>
<li><a href="60ecb00dd7https://redirect.github.com/rubocop/rubocop/issues/13328 ">#13328</a>] Declare <code>Enabled</code> as a common config key</li>
<li><a href="99022d9880https://redirect.github.com/rubocop/rubocop/issues/13327 ">#13327</a> from koic/make_server_mode_aware_of_auto_restart_fo...</li>
<li><a href="60432f5e446b31c39f1eadb7ceed43https://redirect.github.com/rubocop/rubocop/issues/9816 ">#9816</a>] Refine <code>Lint/SafeNavigationConsistency</code></li>
<li><a href="7d6797cf9dhttps://redirect.github.com/rubocop/rubocop/issues/13286 ">#13286</a>] Add <code>AllowedMethods</code> and <code>AllowedPatterns</code> configuration to `Lay...</li>
<li>Additional commits viewable in <a href="https://github.com/rubocop/rubocop/compare/v1.66.1...v1.67.0 ">compare view</a></li>
</ul>
</details>
<br />
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores )
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
</details>
2024-10-16 10:56:58 +08:00
322a3be2db
DEV: Remove logical OR assignment of constants ( #29201 )
...
Constants should always be only assigned once. The logical OR assignment
of a constant is a relic of the past before we used zeitwerk for
autoloading and had bugs where a file could be loaded twice resulting in
constant redefinition warnings.
2024-10-16 10:09:07 +08:00
f3f37c9019
Build(deps): Bump actionpack from 7.1.4 to 7.1.4.1 ( #29229 )
...
Bumps [actionpack](https://github.com/rails/rails ) from 7.1.4 to 7.1.4.1.
- [Release notes](https://github.com/rails/rails/releases )
- [Changelog](https://github.com/rails/rails/blob/v7.2.1.1/actionpack/CHANGELOG.md )
- [Commits](https://github.com/rails/rails/compare/v7.1.4...v7.1.4.1 )
---
updated-dependencies:
- dependency-name: actionpack
dependency-type: direct:production
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2024-10-16 09:47:22 +08:00
e33da4e0e4
Build(deps-dev): Bump @swc/core from 1.7.35 to 1.7.36 ( #29220 )
...
Bumps [@swc/core](https://github.com/swc-project/swc ) from 1.7.35 to 1.7.36.
- [Release notes](https://github.com/swc-project/swc/releases )
- [Changelog](https://github.com/swc-project/swc/blob/main/CHANGELOG.md )
- [Commits](https://github.com/swc-project/swc/compare/v1.7.35...v1.7.36 )
---
updated-dependencies:
- dependency-name: "@swc/core"
dependency-type: direct:development
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2024-10-16 07:15:31 +08:00
Roman Rizzi
Loïc Guitaut
David Taylor
David Battersby
Ted Johansson
chapoi
Joffrey JAFFEUX
dependabot[bot]
Jarek Radosz
Bianca Nenciu
Rafael dos Santos Silva
Sam
Régis Hanol
Natalie Tay
Kris
Jordan Vidrine
Isaac Janzen
Juan David Martínez Cubillos
Discourse Translator Bot
Alan Guo Xiang Tan