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

ref: Rename SentryClient and SentryUser #2403

Merged
merged 103 commits into from
Nov 22, 2022
Merged
Show file tree
Hide file tree
Changes from 98 commits
Commits
Show all changes
103 commits
Select commit Hold shift + click to select a range
b0aa62f
Sentry Swift
brustolin Sep 14, 2022
65205f6
Updated references
brustolin Sep 19, 2022
61997a2
Update TypeAlias.swift
brustolin Sep 19, 2022
423aea1
Update project.pbxproj
brustolin Sep 19, 2022
9f08056
Update tests
brustolin Sep 19, 2022
aa39ceb
remove SentryPerformance
brustolin Sep 20, 2022
13379e5
Merge branch 'master' into feat/swift
brustolin Sep 20, 2022
0911615
Update NotificationCenterTestCase.swift
brustolin Sep 20, 2022
96c0aec
wip
brustolin Sep 20, 2022
18f0139
Update NotificationCenterTestCase.swift
brustolin Sep 20, 2022
a2f875e
Merge branch 'master' into feat/swift
brustolin Sep 20, 2022
ac8c97c
Objc progress
brustolin Sep 20, 2022
624e335
change to framework
brustolin Sep 21, 2022
75e1930
more work
brustolin Sep 21, 2022
a2bb941
Update Sentry.xcscheme
brustolin Sep 21, 2022
469993c
WatchOs Compile
brustolin Sep 21, 2022
e80bd68
update dependency
brustolin Sep 21, 2022
def3389
Merge branch 'master' into feat/swift
brustolin Sep 21, 2022
af05fc7
Auto sign and distribution
philipphofmann Sep 22, 2022
b725721
Config file
brustolin Sep 22, 2022
1f07f60
Merge branch 'feat/swift' of https://github.com/getsentry/sentry-coco…
brustolin Sep 22, 2022
22d6749
Merge branch 'master' into feat/swift
brustolin Sep 22, 2022
e1eb332
some update
brustolin Sep 22, 2022
4fca18c
Merge branch 'feat/swift' of https://github.com/getsentry/sentry-coco…
brustolin Sep 22, 2022
1261c72
framework sample
brustolin Sep 22, 2022
139aae6
Update project.pbxproj
brustolin Sep 22, 2022
09d2e49
Update Makefile
brustolin Sep 22, 2022
ee39ebd
Merge branch 'master' into feat/swift
brustolin Sep 23, 2022
6f5d035
Update Sentry.podspec
brustolin Sep 29, 2022
fa902d2
Merge branch 'master' into feat/swift
brustolin Sep 29, 2022
45100a6
update podspec
brustolin Sep 30, 2022
51ca606
some fix
brustolin Sep 30, 2022
5420a72
Cocoa pod should work
brustolin Sep 30, 2022
2c7a928
Fix tests
brustolin Sep 30, 2022
18a4900
Adding Sentry header
brustolin Oct 3, 2022
e80fadc
Update Sentry.podspec
brustolin Oct 3, 2022
99f7dc7
Update add-sentry-to-vlc.patch
brustolin Oct 3, 2022
18d0be6
Reverting SentryObjc.h to Sentry.h
brustolin Oct 3, 2022
d4ebaed
Yeap, we need SentryObjc
brustolin Oct 3, 2022
516fdfd
tests
brustolin Oct 3, 2022
e9cc793
Define Sentry.modulemap
brustolin Oct 3, 2022
8381895
Merge branch 'master' into feat/swift
brustolin Oct 10, 2022
3479fe1
revert
brustolin Oct 13, 2022
a38db37
Merge branch 'master' into feat/swift
brustolin Oct 13, 2022
25386e4
All over again
brustolin Oct 14, 2022
add046c
Tests
brustolin Oct 14, 2022
6020920
Merge branch 'master' into feat/swift
brustolin Oct 14, 2022
04253dc
Update Sentry.podspec
brustolin Oct 14, 2022
8606b19
pod update
brustolin Oct 14, 2022
21b0e2d
Update project.pbxproj
brustolin Oct 14, 2022
f69e00f
Update project.pbxproj
brustolin Oct 14, 2022
1c16be6
gr
brustolin Oct 17, 2022
7876424
brustolin
brustolin Oct 17, 2022
3c3fbb2
cocoapods fix
brustolin Oct 17, 2022
2ecb347
Reorganize things
brustolin Oct 17, 2022
584436f
Update Sentry.podspec
brustolin Oct 18, 2022
c79c611
Merge branch 'master' into feat/swift
brustolin Oct 20, 2022
19e777c
Merge branch 'master' into feat/swift
brustolin Oct 21, 2022
0b0a1f4
updating meta
brustolin Oct 21, 2022
221c860
More information
brustolin Oct 24, 2022
9e7c555
Update CHANGELOG.md
brustolin Oct 24, 2022
7d5e7d1
Merge branch 'master' into feat/swift
brustolin Oct 24, 2022
097f3d1
Apply suggestions from code review
brustolin Oct 25, 2022
7df3dae
docs
brustolin Oct 25, 2022
e1d01e0
Remove ViewControllerSanitizer
brustolin Oct 25, 2022
27bd089
Merge branch 'master' into feat/swift
brustolin Oct 25, 2022
40552f5
Update CHANGELOG.md
brustolin Oct 25, 2022
fc4efab
Merge branch 'master' into feat/swift
brustolin Nov 4, 2022
ad71f3d
Merge branch 'master' into feat/swift
brustolin Nov 8, 2022
21d170d
Update project.pbxproj
brustolin Nov 8, 2022
7376f54
SentryPrivate
brustolin Nov 10, 2022
42d7e3f
Merge branch 'master' into feat/swift
brustolin Nov 10, 2022
e7f1c9d
Update project.pbxproj
brustolin Nov 10, 2022
d73af80
Update project.pbxproj
brustolin Nov 10, 2022
05ee747
Update SentryPrivate.podspec
brustolin Nov 11, 2022
24d6027
Update SentryPrivate.podspec
brustolin Nov 11, 2022
a7bd4c9
pod lint working
brustolin Nov 11, 2022
20dbe28
Fix tests
brustolin Nov 11, 2022
ce7dd6a
Update integration tests
brustolin Nov 11, 2022
c520394
Docs
brustolin Nov 14, 2022
ed70d47
Update add-sentry-to-homekit.patch
brustolin Nov 14, 2022
d1aca42
Merge branch 'master' into feat/swift
brustolin Nov 14, 2022
596825a
Update add-sentry-to-homekit.patch
brustolin Nov 14, 2022
89a6e51
Merge branch 'feat/swift' of https://github.com/getsentry/sentry-coco…
brustolin Nov 14, 2022
d4be11a
Update add-sentry-to-homekit.patch
brustolin Nov 14, 2022
dbeb0a0
Update Sources/Sentry/include/SentrySwift.h
brustolin Nov 14, 2022
8be92cd
Format code
getsentry-bot Nov 14, 2022
347af03
Update main.swift
brustolin Nov 14, 2022
0e90f96
Merge branch 'feat/swift' of https://github.com/getsentry/sentry-coco…
brustolin Nov 14, 2022
58b4b71
Merge branch 'master' into feat/swift
brustolin Nov 14, 2022
c32dbe7
Update Makefile
brustolin Nov 15, 2022
5572e3a
Merge branch 'master' into feat/swift
brustolin Nov 15, 2022
ff2996f
Merge branch 'master' into feat/swift
brustolin Nov 15, 2022
3c2e914
Update SentryPrivate.podspec
brustolin Nov 16, 2022
76d6d1d
Update Sentry.podspec
brustolin Nov 16, 2022
11a5156
Update project.pbxproj
brustolin Nov 17, 2022
1ca0a32
Rename SentryClient and SentryUser
brustolin Nov 17, 2022
7c95fce
Update CHANGELOG.md
brustolin Nov 18, 2022
1b4b338
Update CHANGELOG.md
brustolin Nov 18, 2022
1322a1e
Update CHANGELOG.md
brustolin Nov 21, 2022
5a09cc5
Merge branch '8.0.0' into ref/rename-classes
kevinrenskers Nov 22, 2022
cefabbf
Remove the recovered references.. again
kevinrenskers Nov 22, 2022
ffc2516
Remove file
kevinrenskers Nov 22, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 17 additions & 3 deletions .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@ jobs:
- run: ./scripts/ci-select-xcode.sh
- run: make analyze

validate-podspec:
name: Validate Podspec
validate-podspec-framework:
name: Validate Podspec as Framework
runs-on: macos-12
strategy:
matrix:
Expand All @@ -50,7 +50,21 @@ jobs:
- uses: actions/checkout@v3
- run: ./scripts/ci-select-xcode.sh
- name: Validate Podspec
run: pod lib lint --verbose --platforms=${{ matrix.platform }}
run: pod lib lint --verbose --platforms=${{ matrix.platform }} Sentry.podspec --include-podspecs=SentryPrivate.podspec
shell: sh

validate-podspec-static:
name: Validate Podspec as Static library
runs-on: macos-12
strategy:
matrix:
platform: ['ios', 'macos', 'tvos', 'watchos']

steps:
- uses: actions/checkout@v3
- run: ./scripts/ci-select-xcode.sh
- name: Validate Podspec
run: pod lib lint --verbose --platforms=${{ matrix.platform }} Sentry.podspec --include-podspecs=SentryPrivate.podspec --use-libraries
shell: sh

validate-high-risk-files:
Expand Down
9 changes: 9 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,14 @@
# Changelog

## Unreleased

This version introduces a dependency on Swift, which only impacts you if you don't use Swift. If your project uses Swift already, this change will not affect you.

### Features

- Properly demangle Swift class name (#2162)
- Renaming Client and User to SentryClient and SentryUser (#2403)
brustolin marked this conversation as resolved.
Show resolved Hide resolved

## 7.31.0

### Features
Expand Down
1 change: 1 addition & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -108,3 +108,4 @@ git-commit-add:

release-pod:
pod trunk push Sentry.podspec
pod trunk push SentryPrivate.podspec
7 changes: 7 additions & 0 deletions Package.swift
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,15 @@ let package = Package(
.library(name: "Sentry-Dynamic", type: .dynamic, targets: ["Sentry"])
],
targets: [
.target( name: "SentryPrivate",
path: "Sources",
sources: [
"Swift"
]
),
.target(
name: "Sentry",
dependencies: ["SentryPrivate"],
path: "Sources",
sources: [
"Sentry/",
Expand Down
2 changes: 1 addition & 1 deletion Samples/Carthage-Validation/Framework/input.xcfilelist
Original file line number Diff line number Diff line change
@@ -1 +1 @@
$(SRCROOT)/Carthage/Build/iOS/Sentry.framework
$(SRCROOT)/Carthage/Build/iOS/Sentry.framework
2 changes: 1 addition & 1 deletion Samples/Carthage-Validation/Framework/output.xcfilelist
Original file line number Diff line number Diff line change
@@ -1 +1 @@
$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/Sentry.framework
$(BUILT_PRODUCTS_DIR)/$(FRAMEWORKS_FOLDER_PATH)/Sentry.framework
15 changes: 15 additions & 0 deletions Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj
Original file line number Diff line number Diff line change
Expand Up @@ -132,6 +132,13 @@
remoteGlobalIDString = 637AFDA5243B02760034958B;
remoteInfo = "iOS-Swift";
};
D81A3499291D0B2C005A27A9 /* PBXContainerItemProxy */ = {
isa = PBXContainerItemProxy;
containerPortal = 6308532C2440C44F00DDE4CE /* Sentry.xcodeproj */;
proxyType = 2;
remoteGlobalIDString = D81A3488291D0AC0005A27A9;
remoteInfo = SentryPrivate;
};
D83A30CA279F075800372D0A /* PBXContainerItemProxy */ = {
isa = PBXContainerItemProxy;
containerPortal = 6308532C2440C44F00DDE4CE /* Sentry.xcodeproj */;
Expand Down Expand Up @@ -350,6 +357,7 @@
children = (
630853322440C44F00DDE4CE /* Sentry.framework */,
630853342440C44F00DDE4CE /* SentryTests.xctest */,
D81A349A291D0B2C005A27A9 /* SentryPrivate.framework */,
);
name = Products;
sourceTree = "<group>";
Expand Down Expand Up @@ -720,6 +728,13 @@
remoteRef = 630853332440C44F00DDE4CE /* PBXContainerItemProxy */;
sourceTree = BUILT_PRODUCTS_DIR;
};
D81A349A291D0B2C005A27A9 /* SentryPrivate.framework */ = {
isa = PBXReferenceProxy;
fileType = wrapper.framework;
path = SentryPrivate.framework;
remoteRef = D81A3499291D0B2C005A27A9 /* PBXContainerItemProxy */;
sourceTree = BUILT_PRODUCTS_DIR;
};
/* End PBXReferenceProxy section */

/* Begin PBXResourcesBuildPhase section */
Expand Down
4 changes: 2 additions & 2 deletions Samples/iOS-Swift/iOS-Swift/ViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ class ViewController: UIViewController {
scope.setTag(value: "swift", key: "language")
scope.setExtra(value: String(describing: self), key: "currentViewController")

let user = Sentry.User(userId: "1")
let user = SentryUser(userId: "1")
user.email = "tony@example.com"
scope.setUser(user)

Expand All @@ -33,7 +33,7 @@ class ViewController: UIViewController {
}

// Also works
let user = Sentry.User(userId: "1")
let user = SentryUser(userId: "1")
user.email = "tony1@example.com"
SentrySDK.setUser(user)

Expand Down
11 changes: 6 additions & 5 deletions Sentry.podspec
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ Pod::Spec.new do |s|
s.requires_arc = true
s.frameworks = 'Foundation'
s.libraries = 'z', 'c++'
s.swift_versions = "5.5"
s.pod_target_xcconfig = {
'GCC_ENABLE_CPP_EXCEPTIONS' => 'YES',
'CLANG_CXX_LANGUAGE_STANDARD' => 'c++14',
Expand All @@ -26,20 +27,20 @@ Pod::Spec.new do |s|
}

s.default_subspecs = ['Core']
s.dependency "SentryPrivate", "7.31.0"

s.subspec 'Core' do |sp|
sp.source_files = "Sources/Sentry/**/*.{h,hpp,m,mm,c,cpp}",
"Sources/SentryCrash/**/*.{h,hpp,m,mm,c,cpp}"
sp.public_header_files = "Sources/Sentry/Public/*.h"
"Sources/SentryCrash/**/*.{h,hpp,m,mm,c,cpp}", "Sources/Swift/Sentry.swift"
sp.public_header_files =
"Sources/Sentry/Public/*.h"
end

s.subspec 'HybridSDK' do |sp|
sp.source_files = "Sources/Sentry/**/*.{h,hpp,m,mm,c,cpp}",
"Sources/SentryCrash/**/*.{h,hpp,m,mm,c,cpp}"
"Sources/SentryCrash/**/*.{h,hpp,m,mm,c,cpp}", "Sources/Swift/Sentry.swift"

sp.public_header_files =
"Sources/Sentry/Public/*.h", "Sources/Sentry/include/PrivateSentrySDKOnly.h"
end

end
Loading