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

[PLAT-6075] Parse Swift 5.4 fatal error messages #1010

Merged
merged 2 commits into from
Feb 23, 2021

Conversation

nickdowell
Copy link
Contributor

@nickdowell nickdowell commented Feb 23, 2021

Goal

In Swift 5.4 (shipping with iOS 14.5) the format of Swift fatal error messages is changing: swiftlang/swift@1a05171

This PR updates the notifier's parsing code added in #948 to respond to this change.

Changeset

Added new regular expression patterns to cope with the change between versions.

Testing

  • Extended unit tests to cover Swift 5.4 format
  • Verified manually with iOS 14.5 simulator

@github-actions
Copy link

github-actions bot commented Feb 23, 2021

Infer: No issues found 🎉

OCLint: No issues found 🎉

Bugsnag.framework binary size did not change - 1,061,456 bytes

Generated by 🚫 Danger

@nickdowell nickdowell marked this pull request as ready for review February 23, 2021 10:42
@"^(assertion failed|fatal error|precondition failed): ((.+): )?file .+, line \\d+\n$",
// From Swift 4.1: https://github.com/apple/swift/commit/d03a575279cf5c523779ef68f8d7903f09ba901e
@"^(Assertion failed|Fatal error|Precondition failed): ((.+): )?file .+, line \\d+\n$",
// From Swift 5.4: https://github.com/apple/swift/commit/1a051719e3b1b7c37a856684dd037d482fef8e59
Copy link
Contributor

Choose a reason for hiding this comment

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

This update presents this information in a format matching the diagnostic conventions used by the compiler, which can be a slight productivity boost.

hahaha, well it does something nicer in vim at least, but that's an interesting reason to change a programming language 🙃

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