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

Release v6.9.2 #1095

Merged
merged 37 commits into from
May 19, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
9b68a7b
Trigger RN tests for all builds of next
twometresteve Apr 28, 2021
cef196a
Merge pull request #1085 from bugsnag/tms/trigger-rn
twometresteve Apr 28, 2021
59334de
Merge branch 'master' into next
nickdowell May 4, 2021
1bee854
Declare BSGBreadcrumbTypeValue and BSGBreadcrumbTypeFromString in Bug…
kstenerud May 5, 2021
1461cf0
Merge pull request #1088 from bugsnag/6370-unity-support
kstenerud May 6, 2021
65f0955
Add BSGInternalErrorReporter
nickdowell May 6, 2021
6db519d
Internal error reporting for BSGEventUploadKSCrashReportOperation
nickdowell May 7, 2021
af27a19
Add internal_error_reporting.feature
nickdowell May 7, 2021
50975ee
Omit user info from internal error reports
nickdowell May 7, 2021
663c836
Remove redundant test code
nickdowell May 10, 2021
0aad729
Don't ignore NSFileReadNoSuchFileError errors
nickdowell May 10, 2021
f58f253
Merge pull request #1089 from bugsnag/nickdowell/internal-error-repor…
nickdowell May 10, 2021
e1f3fd2
Attach appium server logs to MacOS test runs (#1090)
Cawllec May 11, 2021
c97607a
fix: remove failure msg when disabling crash handling (#1091)
kattrali May 12, 2021
0a86ea3
Use Xcode build settings instead of warning flags where possible
nickdowell May 12, 2021
7a28540
Apply xcconfig to targets, not project
nickdowell May 12, 2021
ccf9291
Enable -Wcast-qual
nickdowell May 12, 2021
a4aa6bd
Enable -Wcustom-atomic-properties
nickdowell May 12, 2021
5f257da
Enable -Wdirect-ivar-access
nickdowell May 12, 2021
6bb448b
Enable -Wdocumentation-unknown-command
nickdowell May 12, 2021
0de7f0e
Enable -Wformat-nonliteral
nickdowell May 12, 2021
72d6940
Enable -Widiomatic-parentheses
nickdowell May 12, 2021
6fe3e6f
Enable -Wimport-preprocessor-directive-pedantic
nickdowell May 12, 2021
564d56b
Enable -Wmissing-variable-declarations
nickdowell May 12, 2021
bbeba85
Enable -Woverriding-method-mismatch
nickdowell May 12, 2021
7a4882f
Enable -Wswitch-enum
nickdowell May 12, 2021
8760add
Enable -Wunused-macros
nickdowell May 12, 2021
ca484b6
Add comments about warning flags not enabled
nickdowell May 12, 2021
30c9675
Fix OCLint warning
nickdowell May 12, 2021
6903bce
Remove redundant BSGConfiguration workaround
nickdowell May 13, 2021
741c8d9
Reduce duplication of 0x%lx format string
nickdowell May 13, 2021
697ca1d
Merge pull request #1092 from bugsnag/nickdowell/additional-warnings
nickdowell May 13, 2021
eaa97a7
Disable Cocopods prefix header
nickdowell May 13, 2021
d47e567
Enable additional warnings for fixtures/ios-swift-cocoapods
nickdowell May 13, 2021
4a34d9b
Fix -Wformat build warning
nickdowell May 13, 2021
97f74dc
Merge pull request #1094 from bugsnag/nickdowell/cocoapods-warnings
nickdowell May 13, 2021
dfa33bf
Release v6.9.2
nickdowell May 19, 2021
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
6 changes: 3 additions & 3 deletions .buildkite/pipeline.full.yml
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ steps:
plugins:
artifacts#v1.3.0:
download: ["features/fixtures/macos/output/macOSTestApp.zip"]
upload: ["macOSTestApp.log"]
upload: ["macOSTestApp.log", "appium_server.log"]
commands:
- bundle install
- bundle exec maze-runner
Expand All @@ -107,7 +107,7 @@ steps:
plugins:
artifacts#v1.3.0:
download: ["features/fixtures/macos/output/macOSTestApp.zip"]
upload: ["macOSTestApp.log"]
upload: ["macOSTestApp.log", "appium_server.log"]
commands:
- bundle install
- bundle exec maze-runner
Expand All @@ -127,7 +127,7 @@ steps:
plugins:
artifacts#v1.3.0:
download: ["features/fixtures/macos/output/macOSTestApp.zip"]
upload: ["macOSTestApp.log"]
upload: ["macOSTestApp.log", "appium_server.log"]
commands:
- bundle install
- bundle exec maze-runner
Expand Down
2 changes: 1 addition & 1 deletion .buildkite/pipeline.quick.yml
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ steps:
plugins:
artifacts#v1.3.0:
download: ["features/fixtures/macos/output/macOSTestApp.zip"]
upload: ["macOSTestApp.log"]
upload: ["macOSTestApp.log", "appium_server.log"]
commands:
- bundle install
- bundle exec maze-runner
Expand Down
23 changes: 20 additions & 3 deletions .buildkite/pipeline.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,17 @@
env:
LANG: "en_GB.UTF-8"

steps:
- label: 'Trigger RN tests for all builds of our next branch'
if: build.branch == "next"
trigger: 'bugsnag-js'
build:
branch: 'next'
message: 'Run RN tests with latest Cocoa next branch'
env:
BUILD_RN_WITH_LATEST_NATIVES: "true"
async: true

- label: Build cocoa IPA
key: cocoa_fixture
timeout_in_minutes: 20
Expand Down Expand Up @@ -186,7 +199,7 @@ steps:
plugins:
artifacts#v1.3.0:
download: ["features/fixtures/macos/output/macOSTestApp.zip"]
upload: ["macOSTestApp.log"]
upload: ["macOSTestApp.log", "appium_server.log"]
commands:
- bundle install
- bundle exec maze-runner
Expand All @@ -205,13 +218,17 @@ steps:
env:
STRESS_TEST: "true"
commands:
- echo "--- Bundle install"
- bundle install
- echo "--- Build"
- make -C features/fixtures/macos-stress-test
- echo "--- Test"
- bundle exec maze-runner
features/stress_test.feature
--no-log-requests
artifact_paths:
- features/fixtures/macos-stress-test/BugsnagStressTest.stdout.log
- features/fixtures/macos-stress-test/BugsnagStressTest.stderr.log
- features/fixtures/macos-stress-test/*.log
- features/fixtures/macos-stress-test/*.crash

- label: 'Conditionally trigger full set of tests'
command: sh -c .buildkite/pipeline_trigger.sh
4 changes: 2 additions & 2 deletions .jazzy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@ author_url: "https://www.bugsnag.com"
author: "Bugsnag Inc"
clean: false # avoid deleting docs/.git
framework_root: "Bugsnag"
github_file_prefix: "https://github.com/bugsnag/bugsnag-cocoa/tree/v6.9.1/Bugsnag"
github_file_prefix: "https://github.com/bugsnag/bugsnag-cocoa/tree/v6.9.2/Bugsnag"
github_url: "https://github.com/bugsnag/bugsnag-cocoa"
hide_documentation_coverage: true
module: "Bugsnag"
module_version: "6.9.1"
module_version: "6.9.2"
objc: true
output: "docs"
readme: "README.md"
Expand Down
5 changes: 3 additions & 2 deletions Bugsnag.podspec.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "Bugsnag",
"version": "6.9.1",
"version": "6.9.2",
"summary": "The Bugsnag crash reporting framework for Apple platforms.",
"homepage": "https://bugsnag.com",
"license": "MIT",
Expand All @@ -9,7 +9,7 @@
},
"source": {
"git": "https://github.com/bugsnag/bugsnag-cocoa.git",
"tag": "v6.9.1"
"tag": "v6.9.2"
},
"frameworks": [
"Foundation",
Expand All @@ -28,6 +28,7 @@
"Bugsnag/{**/,}*.{m,h,mm,c}"
],
"requires_arc": true,
"prefix_header_file": false,
"public_header_files": [
"Bugsnag/include/Bugsnag/*.h"
]
Expand Down
116 changes: 98 additions & 18 deletions Bugsnag.xcconfig
Original file line number Diff line number Diff line change
@@ -1,19 +1,99 @@

GCC_TREAT_WARNINGS_AS_ERRORS = YES

WARNING_CFLAGS = $(inherited) -Warc-repeated-use-of-weak -Wassign-enum
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wblock-capture-autoreleasing -Wbool-conversion
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wcomma -Wconditional-uninitialized -Wconstant-conversion -Wconversion
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wdeprecated-declarations -Wdeprecated-implementations -Wdeprecated-objc-isa-usage -Wdocumentation -Wduplicate-method-match
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wempty-body -Wenum-conversion -Wexplicit-ownership-type
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wfloat-conversion -Wfour-char-constants -Wframework-include-private-from-public
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wimplicit-atomic-properties -Wimplicit-int-float-conversion -Wimplicit-retain-self -Wincompatible-pointer-types
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wincomplete-implementation -Winfinite-recursion -Wint-conversion
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wmissing-braces
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wno-unknown-warning-option -Wnon-literal-null-conversion -Wnonportable-include-path -Wnullable-to-nonnull-conversion
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wobjc-literal-conversion -Wobjc-root-class
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wparentheses -Wpointer-sign -Wprotocol
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wreturn-type
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wshadow -Wshorten-64-to-32 -Wsign-compare -Wsign-conversion -Wstrict-prototypes -Wstrict-selector-match -Wswitch
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wundeclared-selector -Wunguarded-availability -Wuninitialized -Wunknown-pragmas -Wunreachable-code
WARNING_CFLAGS = $(WARNING_CFLAGS) -Wunused-function -Wunused-label -Wunused-parameter -Wunused-value -Wunused-variable
// Apple Clang - Warning Policies

GCC_TREAT_WARNINGS_AS_ERRORS = YES // -Werror
GCC_WARN_INHIBIT_ALL_WARNINGS = NO
GCC_WARN_PEDANTIC = NO

// Apple Clang - Warnings - All languages

CLANG_WARN_ASSIGN_ENUM = YES // -Wassign-enum
CLANG_WARN_BLOCK_CAPTURE_AUTORELEASING = YES // -Wblock-capture-autoreleasing
CLANG_WARN_BOOL_CONVERSION = YES // -Wbool-conversion
CLANG_WARN_COMMA = YES // -Wcomma
CLANG_WARN_CONSTANT_CONVERSION = YES // -Wconstant-conversion
CLANG_WARN_DOCUMENTATION_COMMENTS = YES // -Wdocumentation
CLANG_WARN_EMPTY_BODY = YES // -Wempty-body
CLANG_WARN_ENUM_CONVERSION = YES // -Wenum-conversion
CLANG_WARN_FLOAT_CONVERSION = YES // -Wfloat-conversion
CLANG_WARN_FRAMEWORK_INCLUDE_PRIVATE_FROM_PUBLIC = YES // -Wframework-include-private-from-public
CLANG_WARN_IMPLICIT_SIGN_CONVERSION = YES // -Wsign-conversion
CLANG_WARN_INFINITE_RECURSION = YES // -Winfinite-recursion
CLANG_WARN_INT_CONVERSION = YES // -Wint-conversion
CLANG_WARN_NON_LITERAL_NULL_CONVERSION = YES // -Wnon-literal-null-conversion
CLANG_WARN_PRAGMA_PACK = YES // -Wpragma-pack
CLANG_WARN_PRIVATE_MODULE = YES // -Wprivate-module
CLANG_WARN_QUOTED_INCLUDE_IN_FRAMEWORK_HEADER = YES // -Wquoted-include-in-framework-header
CLANG_WARN_SEMICOLON_BEFORE_METHOD_BODY = YES // -Wsemicolon-before-method-body
CLANG_WARN_STRICT_PROTOTYPES = YES // -Wstrict-prototypes
CLANG_WARN_SUSPICIOUS_IMPLICIT_CONVERSION = YES // -Wconversion -Wconstant-conversion -Wint-conversion -Wbool-conversion -Wenum-conversion -Wfloat-conversion -Wnon-literal-null-conversion -Wobjc-literal-conversion
CLANG_WARN_UNGUARDED_AVAILABILITY = YES_AGGRESSIVE // -Wunguarded-availability
CLANG_WARN_UNREACHABLE_CODE = YES // -Wunreachable-code
GCC_TREAT_IMPLICIT_FUNCTION_DECLARATIONS_AS_ERRORS = YES
GCC_TREAT_INCOMPATIBLE_POINTER_TYPE_WARNINGS_AS_ERRORS = YES // -Wincompatible-pointer-types
GCC_WARN_64_TO_32_BIT_CONVERSION = YES // -Wshorten-64-to-32
GCC_WARN_ABOUT_DEPRECATED_FUNCTIONS = YES // -Wdeprecated-declarations
GCC_WARN_ABOUT_MISSING_FIELD_INITIALIZERS = NO // -Wmissing-field-initializers
GCC_WARN_ABOUT_MISSING_NEWLINE = YES // -Wnewline-eof
GCC_WARN_ABOUT_MISSING_PROTOTYPES = NO // -Wmissing-prototypes
GCC_WARN_ABOUT_POINTER_SIGNEDNESS = YES
GCC_WARN_ABOUT_RETURN_TYPE = YES_ERROR // -Wreturn-type
GCC_WARN_CHECK_SWITCH_STATEMENTS = YES // -Wswitch
GCC_WARN_FOUR_CHARACTER_CONSTANTS = YES // -Wfour-char-constants
GCC_WARN_INITIALIZER_NOT_FULLY_BRACKETED = YES // -Wmissing-braces
GCC_WARN_MISSING_PARENTHESES = YES // -Wparentheses
GCC_WARN_SHADOW = YES // -Wshadow
GCC_WARN_SIGN_COMPARE = YES // -Wsign-compare
GCC_WARN_TYPECHECK_CALLS_TO_PRINTF = YES // -Wformat
GCC_WARN_UNINITIALIZED_AUTOS = YES_AGGRESSIVE // -Wuninitialized
GCC_WARN_UNKNOWN_PRAGMAS = YES // -Wunknown-pragmas
GCC_WARN_UNUSED_FUNCTION = YES // -Wunused-function
GCC_WARN_UNUSED_LABEL = YES // -Wunused-label
GCC_WARN_UNUSED_PARAMETER = YES // -Wunused-parameter
GCC_WARN_UNUSED_VALUE = YES // -Wunused-value
GCC_WARN_UNUSED_VARIABLE = YES // -Wunused-variable

// Apple Clang - Warnings - C++

CLANG_WARN_ATOMIC_IMPLICIT_SEQ_CST = YES // -Watomic-implicit-seq-cst
CLANG_WARN_CXX0X_EXTENSIONS = YES // -Wc++11-extensions
CLANG_WARN_DELETE_NON_VIRTUAL_DTOR = YES // -Wdelete-non-virtual-dtor
CLANG_WARN_RANGE_LOOP_ANALYSIS = YES // -Wrange-loop-analysis
CLANG_WARN_SUSPICIOUS_MOVE = YES // -Wmove
CLANG_WARN_VEXING_PARSE = YES // -Wvexing-parse
CLANG_WARN__EXIT_TIME_DESTRUCTORS = YES // -Wexit-time-destructors
GCC_WARN_ABOUT_INVALID_OFFSETOF_MACRO = YES // -Winvalid-offsetof
GCC_WARN_HIDDEN_VIRTUAL_FUNCTIONS = YES // -Woverloaded-virtual
GCC_WARN_NON_VIRTUAL_DESTRUCTOR = YES // -Wnon-virtual-dtor

// Apple Clang - Warnings - Objective-C

CLANG_WARN_DEPRECATED_OBJC_IMPLEMENTATIONS = YES // -Wdeprecated-implementations
CLANG_WARN_DIRECT_OBJC_ISA_USAGE = YES // -Wdeprecated-objc-isa-usage
CLANG_WARN_MISSING_NOESCAPE = YES // -Wmissing-noescape
CLANG_WARN_OBJC_IMPLICIT_ATOMIC_PROPERTIES = YES // -Wimplicit-atomic-properties
CLANG_WARN_OBJC_INTERFACE_IVARS = YES // -Wobjc-interface-ivars
CLANG_WARN_OBJC_LITERAL_CONVERSION = YES // -Wobjc-literal-conversion
CLANG_WARN_OBJC_MISSING_PROPERTY_SYNTHESIS = NO // -Wobjc-missing-property-synthesis
CLANG_WARN_OBJC_ROOT_CLASS = YES // -Wobjc-root-class
CLANG_WARN__DUPLICATE_METHOD_MATCH = YES // -Wduplicate-method-match
GCC_WARN_ALLOW_INCOMPLETE_PROTOCOL = YES // -Wprotocol
GCC_WARN_STRICT_SELECTOR_MATCH = YES // -Wstrict-selector-match
GCC_WARN_UNDECLARED_SELECTOR = YES // -Wundeclared-selector

// Apple Clang - Warnings - Objective-C and ARC

CLANG_WARN_OBJC_EXPLICIT_OWNERSHIP_TYPE = YES // -Wexplicit-ownership-type
CLANG_WARN_OBJC_IMPLICIT_RETAIN_SELF = YES // -Wimplicit-retain-self
CLANG_WARN_OBJC_REPEATED_USE_OF_WEAK = YES // -Warc-repeated-use-of-weak -Wno-arc-maybe-repeated-use-of-weak
CLANG_WARN__ARC_BRIDGE_CAST_NONARC = YES // -Warc-bridge-casts-disallowed-in-nonarc

// Warning flags that have no dedicated Xcode build settings

WARNING_CFLAGS = -Wcast-qual -Wconditional-uninitialized -Wcustom-atomic-properties -Wdirect-ivar-access -Wdocumentation-unknown-command -Wformat-nonliteral -Widiomatic-parentheses -Wimplicit-int-float-conversion -Wimport-preprocessor-directive-pedantic -Wincomplete-implementation -Wmissing-variable-declarations -Wno-unknown-warning-option -Wnonportable-include-path -Wnullable-to-nonnull-conversion -Woverriding-method-mismatch -Wpointer-sign -Wswitch-enum -Wunused-macros

// Some flags that were considered and rejected:
//
// -Wcstring-format-directive warns when a %s format string is used to construct a CF/NSString, despite them being supported.
// -Wpadded warns when struct padding is added by the compiler, a common occurrence and not a programming error.
//
Loading