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

Clean up detect-changes.yaml #257

Merged
merged 3 commits into from
Oct 13, 2023
Merged

Conversation

djorgensendk
Copy link
Contributor

No description provided.

@cla-bot cla-bot bot added the cla-signed label Oct 11, 2023
@djorgensendk djorgensendk temporarily deployed to AzureAuth October 11, 2023 12:33 — with GitHub Actions Inactive
@djorgensendk djorgensendk temporarily deployed to AzureAuth October 11, 2023 12:33 — with GitHub Actions Inactive
@codecov-commenter
Copy link

codecov-commenter commented Oct 11, 2023

Codecov Report

Merging #257 (c306131) into main (fdd5648) will decrease coverage by 6.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #257      +/-   ##
==========================================
- Coverage   76.55%   69.87%   -6.68%     
==========================================
  Files          23      210     +187     
  Lines         435     3771    +3336     
  Branches       37      441     +404     
==========================================
+ Hits          333     2635    +2302     
- Misses         89      986     +897     
- Partials       13      150     +137     
Flag Coverage Δ
appcommon 61.83% <ø> (?)
databricks 76.55% <ø> (ø)
jsonserialization 36.66% <ø> (?)
logging 45.23% <ø> (?)
messaging 51.42% <ø> (?)
testcommon 73.72% <ø> (?)

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

see 187 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link

github-actions bot commented Oct 11, 2023

Test Results

6 tests   6 ✔️  18s ⏱️
1 suites  0 💤
1 files    0

Results for commit c306131.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 11, 2023

Test results for ./source/TestCommon/TestCommon.sln

    2 files      2 suites   4m 20s ⏱️
124 tests 124 ✔️ 0 💤 0
126 runs  126 ✔️ 0 💤 0

Results for commit dfacb27.

♻️ This comment has been updated with latest results.

@djorgensendk djorgensendk temporarily deployed to AzureAuth October 12, 2023 09:17 — with GitHub Actions Inactive
@djorgensendk djorgensendk temporarily deployed to AzureAuth October 12, 2023 09:17 — with GitHub Actions Inactive
@djorgensendk djorgensendk temporarily deployed to AzureAuth October 12, 2023 09:17 — with GitHub Actions Inactive
Copy link
Contributor

@dstenroejl dstenroejl left a comment

Choose a reason for hiding this comment

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

Added two comments

@djorgensendk djorgensendk temporarily deployed to AzureAuth October 12, 2023 11:34 — with GitHub Actions Inactive
@djorgensendk djorgensendk temporarily deployed to AzureAuth October 12, 2023 11:34 — with GitHub Actions Inactive
@djorgensendk djorgensendk temporarily deployed to AzureAuth October 12, 2023 11:34 — with GitHub Actions Inactive
@djorgensendk djorgensendk merged commit 21baea1 into main Oct 13, 2023
31 checks passed
@djorgensendk djorgensendk deleted the djorgensen/clean-up-detect-changes branch October 13, 2023 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants