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

Moved some stuff from Travis to GitHub Actions / Cleanups #3715

Merged
merged 15 commits into from
May 31, 2022

Conversation

firewave
Copy link
Collaborator

No description provided.

@firewave
Copy link
Collaborator Author

@danmar Could you please take a look at the MISRA failure/output? Thanks.

@danmar
Copy link
Owner

danmar commented Jan 18, 2022

@firewave ok I will..

@danmar
Copy link
Owner

danmar commented Jan 18, 2022

@firewave I hope I fixed it

@firewave
Copy link
Collaborator Author

@danmar Thanks, that helped. But now some other MISRA test fails.

@firewave
Copy link
Collaborator Author

firewave commented Feb 2, 2022

@danmar There's still a MISRA test failure. Please have another look. Thanks.

@firewave
Copy link
Collaborator Author

@danmar Please have a look at the MISRA test failure. Thanks.

@firewave
Copy link
Collaborator Author

@danmar Sorry for being pushy but this still needs MISRA attention and I have no clue about that.

@firewave
Copy link
Collaborator Author

firewave commented May 9, 2022

There's something wrong with the exitcode on Linux now since the tests pass there although they fail on Windows.

@firewave
Copy link
Collaborator Author

firewave commented May 9, 2022

@danmar There's a test failure with the naming addon. Please have a look.

@firewave
Copy link
Collaborator Author

There's something wrong with the exitcode on Linux now since the tests pass there although they fail on Windows.

Turns out it is the test. naming_test.c fails on Windows but works fine on Linux. I disabled it for now.

@firewave firewave marked this pull request as ready for review May 26, 2022 16:41
@firewave
Copy link
Collaborator Author

@danmar This is finally ready for review.

@danmar danmar merged commit 5e2afac into danmar:main May 31, 2022
@firewave firewave deleted the travis branch May 31, 2022 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants