pocket-casts-ios icon indicating copy to clipboard operation
pocket-casts-ios copied to clipboard

Address a couple of warnings

Open mokagio opened this issue 2 years ago • 0 comments

Removes an unused variable and a nil-coalescing for a non-optional value.

To test

If CI is green, this is safe to merge.

Checklist

  • [x] I have considered if this change warrants user-facing release notes and have added them to CHANGELOG.md if necessary. – N.A.
  • [x] I have considered adding unit tests for my changes. – N.A.
  • [x] I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics. – N.A.

Notice this PR comes from a fork of the repo. The purpose of this PR is to verify my assumption that Buildkite won't run on this PR, because we don't run it (or can't run it?) against external contributor PR.

One tell tale is that there are no "Details" links in the GitHub checks. Usually, when we create a PR, a Buildkite build starts as well and, even if the build doesn't run for a while in cases where the queue is full, we always have a link to the build in there.

image

mokagio avatar Oct 21 '22 01:10 mokagio