Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

disable trailing_comma #434

Merged
merged 1 commit into from
Jul 26, 2023
Merged

disable trailing_comma #434

merged 1 commit into from
Jul 26, 2023

Conversation

mallexxx
Copy link
Collaborator

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL:
What kind of version bump will this require?: -

Description:

  • disable trailing_comma rule

Steps to test this PR:

  1. validate swiftlint lint doesn‘t raise config validation issues

@mallexxx mallexxx requested a review from Bunn July 26, 2023 06:38
Bunn
Bunn approved these changes Jul 26, 2023
View reviewed changes
Copy link
Contributor

@Bunn Bunn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@mallexxx mallexxx merged commit 51c9619 into main Jul 26, 2023
@mallexxx mallexxx deleted the alex/disable-trailing_comma branch July 26, 2023 10:48
samsymons added a commit that referenced this pull request Jul 27, 2023
@samsymons
Merge branch 'main' into sam/secure-storage
e390500 
* main:
  Fire Window (#420)
  disable trailing_comma (#434)
  Add support for per-model Sync Data Providers initialization (#433)
afterxleep pushed a commit that referenced this pull request Aug 2, 2023
@mallexxx @afterxleep
disable trailing_comma (#434)
9f5fb53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Bunn Bunn Bunn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mallexxx @Bunn