Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swiftlint refactoring #882

Merged
merged 3 commits into from
Jul 9, 2024
Merged

Swiftlint refactoring #882

merged 3 commits into from
Jul 9, 2024

Conversation

federicocappelli
Copy link
Member

Task/Issue URL: https://app.asana.com/0/1205842942115003/1207743897886423/f
What kind of version bump will this require?: No version bump

Description:

Swiftlint rules disabled:

  • line_length
  • function_parameter_count
  • function_body_length
  • file_length
  • type_body_length removed

cyclomatic_complexity disabled only for switch cases and superfluous disable removed


Internal references:

Software Engineering Expectations
Technical Design Template

@afterxleep afterxleep self-requested a review July 9, 2024 12:52
@federicocappelli federicocappelli merged commit dd6df53 into main Jul 9, 2024
9 checks passed
@federicocappelli federicocappelli deleted the fcappelli/swiftlint branch July 9, 2024 13:31
samsymons added a commit that referenced this pull request Jul 19, 2024
* main:
  Remove unused VPN session utilities (#898)
  Add new deprecated Mac remote message attribute. (#903)
  Resetting all state for the VPN will cancel the tunnel and stop the monitors (#900)
  Add support for skipping sending usage pixels for remote messages (#902)
  Bump Tests/BrowserServicesKitTests/Resources/privacy-reference-tests (#896)
  Removes the listen port from the wireguard client (#901)
  Be explicit when performing developer redirects (#884)
  C-S-S cross origin fixes
  Update C-S-S version (#892)
  Add a debug menu action to reset Remote Messages on macOS (#891)
  Add desktop specific RMF attributes (#883)
  Upload exception message to Sentry (#856)
  Add locale to broken site report (#889)
  Add new subfeature for duckplayer (#885)
  Swiftlint refactoring (#882)
  Remote Messaging Framework for macOS (#876)
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.

2 participants