Implement packet capture client and models #4991
niklasberglundAnnotations
20 warnings and 1 notice
|
Run swiftlint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run swiftlint
swiftlint 0.55.1 is already installed and up-to-date.
To reinstall 0.55.1, run:
brew reinstall swiftlint
|
|
Run swiftlint:
PacketTunnel/PacketTunnelProvider/PacketTunnelProvider.swift#L36
Function body should span 50 lines or less excluding comments and whitespace: currently spans 54 lines (function_body_length)
|
|
Run swiftlint:
MullvadSettings/SettingsManager.swift#L51
Prefer using UTF-8 encoded strings when converting between `String` and `Data` (non_optional_string_data_conversion)
|
|
Run swiftlint:
MullvadVPNTests/MullvadSettings/IPOverrideRepositoryTests.swift#L17
Prefer `static` over `final class` (static_over_final_class)
|
|
Run swiftlint:
MullvadVPNTests/MullvadSettings/IPOverrideRepositoryTests.swift#L21
Prefer `static` over `final class` (static_over_final_class)
|
|
Run swiftlint:
MullvadVPNTests/MullvadSettings/MigrationManagerTests.swift#L75
Prefer using UTF-8 encoded strings when converting between `String` and `Data` (non_optional_string_data_conversion)
|
|
Run swiftlint:
MullvadVPNTests/MullvadSettings/MigrationManagerTests.swift#L19
Prefer `static` over `final class` (static_over_final_class)
|
|
Run swiftlint:
MullvadVPNTests/MullvadSettings/MigrationManagerTests.swift#L23
Prefer `static` over `final class` (static_over_final_class)
|
|
Run swiftlint:
MullvadVPNTests/MullvadSettings/APIAccessMethodsTests.swift#L16
Prefer `static` over `final class` (static_over_final_class)
|
|
Run swiftlint:
MullvadVPNTests/MullvadSettings/APIAccessMethodsTests.swift#L20
Prefer `static` over `final class` (static_over_final_class)
|
|
Check formatting
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Check formatting
swiftformat 0.54.0 already installed
|
|
Unit tests
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/cache@v3, actions/setup-go@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Unit tests:
--- xcodebuild#L1
Using the first of multiple matching destinations:
|
|
Unit tests
SwiftLint not installed, download from https://github.com/realm/SwiftLint
|
|
Unit tests:
ld#L1
'/Users/runner/Library/Developer/Xcode/DerivedData/MullvadVPN-eyvrfmadodhkonbxvnjzkuyudpjl/Build/Products/Debug-iphonesimulator/libwg-go.a[arm64][2](go.o)' has malformed LC_DYSYMTAB, expected 61 undefined symbols to start at index 4702, found 73 undefined symbols starting at index 15
|
|
Unit tests
SwiftLint not installed, download from https://github.com/realm/SwiftLint
|
|
Unit tests:
ios/MullvadVPN/Coordinators/ApplicationCoordinator.swift#L261
variable 'nextRoutes' was never mutated; consider changing to 'let' constant
|
|
Unit tests
xcbeautify 2.4.0 is already installed and up-to-date.
To reinstall 2.4.0, run:
brew reinstall xcbeautify
|
|
Unit tests
Tests Passed: 0 failed, 0 skipped, 3 total (0.671 seconds)
|