Skip to content

Conversation

@jrjohnson
Copy link
Member

@jrjohnson jrjohnson commented Sep 30, 2025

We can't use this information in any meaningful way right now because of issues with generating coverage for things in common that are tested in test-app and it's causing build errors that are hard to diagnose. Rid ourselves of this entirely for now.

This will immediately unblock our ember concurrency update in #8823

We can't use this information in any meaningful way right now because of
issues with generating coverage for things in common that are tested in
test-app and it's causing build errors that are hard to diagnose. Rid
ourselves of this entirely for now.
@netlify
Copy link

netlify bot commented Sep 30, 2025

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit 26e9b1d
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/68db5efbfa0ab000080251f9
😎 Deploy Preview https://deploy-preview-8854--ilios-frontend.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@jrjohnson jrjohnson marked this pull request as ready for review September 30, 2025 05:46
Copy link
Member

@stopfstedt stopfstedt left a comment

Choose a reason for hiding this comment

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

YES

@stopfstedt stopfstedt added the run ui tests Run the expensive UI tests label Sep 30, 2025
@stopfstedt stopfstedt merged commit 35089c4 into ilios:master Sep 30, 2025
39 checks passed
@jrjohnson jrjohnson deleted the remove-code-coverage branch September 30, 2025 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run ui tests Run the expensive UI tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants