Skip to content

Remove coverage build code#3478

Merged
stevenengler merged 4 commits intoshadow:mainfrom
stevenengler:remove-coverage
Dec 28, 2024
Merged

Remove coverage build code#3478
stevenengler merged 4 commits intoshadow:mainfrom
stevenengler:remove-coverage

Conversation

@stevenengler
Copy link
Copy Markdown
Contributor

Closes #3433.

@stevenengler stevenengler self-assigned this Dec 24, 2024
@github-actions github-actions bot added Component: Testing Unit and integration tests and frameworks Component: Build Build/install tools and dependencies labels Dec 24, 2024
Copy link
Copy Markdown
Member

@robgjansen robgjansen left a comment

Choose a reason for hiding this comment

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

🎉

Now that it isn't conditional on coverage builds, I think it make more
sense to move this here.
These aren't used anymore (are always empty).
@github-actions github-actions bot added the Component: Documentation In-repository documentation, under docs/ label Dec 28, 2024
@stevenengler
Copy link
Copy Markdown
Contributor Author

I also added a changelog entry.

@stevenengler stevenengler merged commit dab14f4 into shadow:main Dec 28, 2024
@stevenengler stevenengler deleted the remove-coverage branch December 28, 2024 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Build Build/install tools and dependencies Component: Documentation In-repository documentation, under docs/ Component: Testing Unit and integration tests and frameworks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove coverage build

2 participants