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

Updated the CI pipeline #137

Merged
merged 17 commits into from
Mar 20, 2024
Merged

Updated the CI pipeline #137

merged 17 commits into from
Mar 20, 2024

Conversation

leonyi4
Copy link
Contributor

@leonyi4 leonyi4 commented Mar 14, 2024

Please make a new docker repository and add the docker image tag as DOCKER_IMAGE under GitHub variables.

run: |
sudo -s
source /opt/ros/noetic/setup.bash
cd /opt/ros/overlay_ws
Copy link
Collaborator

Choose a reason for hiding this comment

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

What do the first three statements, wouldn't the source, roscd and pytest be enough?

types:
- completed
workflow_dispatch: # manual run
inputs:
Copy link
Collaborator

Choose a reason for hiding this comment

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

The Dockerfile only needs to be updated if the requirements change. We could also automate this: https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#running-your-workflow-only-when-a-push-affects-specific-files

@Tigul Tigul merged commit 06c38d9 into cram2:dev Mar 20, 2024
2 checks passed
@leonyi4 leonyi4 deleted the testing branch March 21, 2024 09:34
hawkina pushed a commit to hawkina/pycram that referenced this pull request Oct 23, 2024
Added demos as packages and adjusted vib-imports
mkhoshnam pushed a commit to mkhoshnam/pycram that referenced this pull request Nov 26, 2024
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