Skip to content

[com1]: call tif check from function #1300

Merged
fso42 merged 1 commit into
masterfrom
testCom8
Jun 29, 2026
Merged

[com1]: call tif check from function #1300
fso42 merged 1 commit into
masterfrom
testCom8

Conversation

@awirb

@awirb awirb commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@awirb awirb requested a review from fso42 June 23, 2026 12:16
@awirb awirb self-assigned this Jun 23, 2026
@qltysh

qltysh Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Qlty


Coverage Impact

⬆️ Merging this pull request will increase total coverage on master by 0.18%.

Modified Components (1)

RatingComponent% Diff
Coverage rating: C Coverage rating: C
com1DFA100.0%

Modified Files with Diff Coverage (1)

RatingFile% DiffUncovered Line #s
Coverage rating: B Coverage rating: B
avaframe/com1DFA/com1DFA.py100.0%
Total100.0%
🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@awirb awirb changed the title call tif check from function [com1]: call tif check from function Jun 23, 2026
@fso42 fso42 added the enhancement New feature or request label Jun 25, 2026

@fso42 fso42 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, only a pytest is missing

@awirb awirb assigned fso42 and unassigned awirb Jun 29, 2026
move to function

use correct parameter

rename

add pytest
@fso42

fso42 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Standardtests ident (apart from known com5)

@fso42 fso42 merged commit 0f3a8b1 into master Jun 29, 2026
4 checks passed
@fso42 fso42 deleted the testCom8 branch June 29, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants