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 GitHub Actions workflow #301

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

robwoolley
Copy link

I saw that you asked for the CI workflow to be updated for #299

I did a quick run at making the necessary changes. The commits are clean however since I was working in the GHA workflow editor some commits failed and had to be corrected. (eg switching from nose to nose3 then to pynose) I am happy to rework the series if you feel it is necessary.

You may see the workflow test runs I did here: https://github.com/robwoolley/superflore/actions

This prevents trailing zeroes from being removed by making the values strings not numbers.
The Ubuntu 22.04 is not supported for ROS.  Temporarily remove it from the matrix until ROS 2 support is added to the workflow.
Replace nose with nose3 which is a "drop-in fork of nose v1 that is compatible with Python 2.7 and Python 3.3-3.10 without using lib2to3 or other compatibility layer"
Use pynose for Python 3.11+ support
Excluding tests for docker and for creating invalid directory paths.  These tests do not seem relevant to testing superflore in a containerized CI/CD pipeline.
Update the checkout and setup-python actions to the latest versions.

Remove Ubuntu 18.04 as a runner image as it has been deprecated by GitHub Actions and no longer as runners available.
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.

1 participant