ci: skip az login except for integration tests

[dciborow]

Description

Testing

Additional context

[github-actions]

Summary by GPT-4

In this diff, the continue-on-error attribute is removed from the Azure/[email protected] action and replaced with an if condition. The action will now only run if the value of matrix.flags is equal to 'integration'. This means that the Azure login step will be skipped for any jobs in the matrix that do not have the 'integration' flag set.

Suggestions

Here are some suggestions for improving the changes in this PR:

1. Add a comment explaining the purpose of the conditional statement:

   # Run Azure login only for integration tests
   if: ${{ matrix.flags == 'integration' }}
   

2. Consider using a more descriptive name for the flags variable in the matrix, such as test_type or test_scope. This would make it clearer what the variable represents and improve readability.

3. Ensure that all necessary dependencies and environment variables are properly set up for both integration and non-integration tests.

4. Make sure to test these changes thoroughly to ensure that they do not introduce any unexpected behavior or break existing functionality.

[codecov-commenter]

Codecov Report

Merging #62 (78936d5) into main (3751927) will decrease coverage by 1.50%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #62      +/-   ##
==========================================
- Coverage   86.78%   85.28%   -1.50%     
==========================================
  Files          13       13              
  Lines         401      401              
  Branches       61       61              
==========================================
- Hits          348      342       -6     
- Misses         37       44       +7     
+ Partials       16       15       -1     

Flag	Coverage Δ
integration	?
unittests	85.28% <ø> (ø)
unittests-3.10	85.28% <ø> (ø)
unittests-3.11	85.28% <ø> (ø)
unittests-3.8	85.28% <ø> (ø)
unittests-3.9	85.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 2 files with indirect coverage changes