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

Update ci to remove warnings #101

Merged
merged 1 commit into from
May 7, 2024

Conversation

mcbarton
Copy link

@mcbarton mcbarton commented May 5, 2024

This PR will remove all warnings which occur when the ci runs.

@mcbarton
Copy link
Author

mcbarton commented May 5, 2024

@vgvassilev @maximusron This PR does what is sets out to do, but has an unintended side effect. It caused the cppyy tests to fail to run on all systems. I can only guess it is do with me update to a newer version of python to stop the pip version error. Any suggestions on how you want to proeed?

@mcbarton
Copy link
Author

mcbarton commented May 5, 2024

Just in case its because the llvm cache was built with version 3.10 and now I have switched to version 3.12 then it may be worth clearing the cache and manually reactivating the workflow.

@mcbarton mcbarton force-pushed the remove-warnings-ci branch from 25a8c27 to e1daed8 Compare May 5, 2024 14:01
@mcbarton mcbarton force-pushed the remove-warnings-ci branch from d068326 to c9774a3 Compare May 5, 2024 18:29
@mcbarton
Copy link
Author

mcbarton commented May 5, 2024

@vgvassilev I managed to fix the issue by changing the python version from 3.12 to 3.11 . @maximusron This does suggest we will get different tests failing to the ones your disabling, for anyone using a later python version.

I consider this PR ready for merging.

@mcbarton
Copy link
Author

mcbarton commented May 7, 2024

@maximusron are you able to merge this?

@vgvassilev vgvassilev merged commit 62117ff into compiler-research:master May 7, 2024
9 of 12 checks passed
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