Remove double negations for checkCancellation function

[Seb-Ltz]

Remove double negations for checkCancellation function

♻️ Current situation & Problem

As mentionned in issue #83 as well as in the PR #81 (discussion), the SpeziLLM package is currently using a double negation (guard followed by !) for every checkCancellation() function call. As double negations make the code harder to read, we could use if instead.

⚙️ Release Notes

• Remove the double negations for checkCancellation by replacing the guard statements that are followed by a not operator into an if statement.

📝 Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

• I agree to follow the Code of Conduct and Contributing Guidelines.

[LeonNissen]

Nice work @Seb-Ltz! LGTM 🚀

[philippzagar]

Looks great @Seb-Ltz, thanks for the quick fix!

[philippzagar]

@Seb-Ltz Seems like the Vision Pro UI tests timed out in the CI (sometimes happens, I retriggered the job, should hopefully pass in the second run.

[philippzagar]

Closes #83

[codecov]

Codecov Report

Attention: Patch coverage is 25.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 39.12%. Comparing base (4a86cbf) to head (2accced).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...urces/SpeziLLMLocal/LLMLocalSession+Generate.swift	0.00%	2 Missing ⚠️
...s/SpeziLLMOpenAI/LLMOpenAISession+Generation.swift	0.00%	2 Missing ⚠️
Sources/SpeziLLMLocal/LLMLocalSession.swift	0.00%	1 Missing ⚠️
Sources/SpeziLLMOpenAI/LLMOpenAISession.swift	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #98   +/-   ##
=======================================
  Coverage   39.12%   39.12%           
=======================================
  Files          64       64           
  Lines        2357     2357           
=======================================
  Hits          922      922           
  Misses       1435     1435           

Files with missing lines	Coverage Δ
Sources/SpeziLLM/Mock/LLMMockSession.swift	78.19% <100.00%> (ø)
Sources/SpeziLLMLocal/LLMLocalSession.swift	0.00% <0.00%> (ø)
Sources/SpeziLLMOpenAI/LLMOpenAISession.swift	22.86% <0.00%> (ø)
...urces/SpeziLLMLocal/LLMLocalSession+Generate.swift	0.00% <0.00%> (ø)
...s/SpeziLLMOpenAI/LLMOpenAISession+Generation.swift	0.00% <0.00%> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a86cbf...2accced. Read the comment docs.