Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Fix coverage#115

Merged
claravanstaden merged 2 commits intosnowbridgefrom
minor-improvements
Feb 22, 2024
Merged

Fix coverage#115
claravanstaden merged 2 commits intosnowbridgefrom
minor-improvements

Conversation

@claravanstaden
Copy link
Copy Markdown
Collaborator

@claravanstaden claravanstaden commented Feb 20, 2024

  • Fixes the code coverage (it didn't run on the default snowbridge branch)
  • Fixes some minor issues with the contribute upstream script

@claravanstaden claravanstaden marked this pull request as ready for review February 20, 2024 10:19
@claravanstaden
Copy link
Copy Markdown
Collaborator Author

Code coverage report is not shown here but it looks like coverage should first run on the default branch (snowbridge) before it will show on PRs.

Copy link
Copy Markdown

@vgeddes vgeddes left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Copy Markdown

@yrong yrong left a comment

Choose a reason for hiding this comment

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

+1

@claravanstaden claravanstaden merged commit 4cede80 into snowbridge Feb 22, 2024
@claravanstaden claravanstaden deleted the minor-improvements branch February 22, 2024 13:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants