Skip to content

Add a test result publishing step #413

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.

Sign up for GitHub

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

Jump to bottom
Merged
jasonleenaylor merged 3 commits into from
Dec 2, 2024
Merged

Add a test result publishing step #413

jasonleenaylor merged 3 commits into from
Dec 2, 2024

Conversation

@jasonleenaylor
Copy link
Contributor

@jasonleenaylor jasonleenaylor commented Nov 27, 2024 •
edited by papeh
Loading

This change is Reviewable

@github-actions
Copy link

github-actions bot commented Nov 27, 2024 •
edited
Loading

Test Results

879 tests   714 ✅  3m 25s ⏱️
108 suites   25 💤
  1 files     20 ❌  120 🔥

For more details on these failures and errors, see this check.

Results for commit a51a96b.

♻️ This comment has been updated with latest results.

hahn-kev
hahn-kev approved these changes Nov 28, 2024
View reviewed changes
Copy link
Contributor

@hahn-kev hahn-kev left a comment

Choose a reason for hiding this comment

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

looks good, I'll ask Robin to take a look at those failing tests and fix them in another PR

@rmunn rmunn mentioned this pull request Nov 29, 2024
Merged
@rmunn
Copy link
Contributor

rmunn commented Nov 29, 2024

#416 should fix the failing LfMergeBridge tests. There are still 120 tests with errors, not failures; most of them appear to be LocalizationManager issues or "System.IO.FileLoadException : Could not load file or assembly 'SIL.Core, Version=14.2.0.0, Culture=neutral, PublicKeyToken=cab3c8c5232dfcf2' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)"

@jasonleenaylor jasonleenaylor merged commit 68d8d61 into develop Dec 2, 2024
1 check passed
@jasonleenaylor jasonleenaylor deleted the chore/reportTests branch December 2, 2024 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hahn-kev hahn-kev hahn-kev approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jasonleenaylor @rmunn @hahn-kev