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.

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

Workflow update - tests, checks #60

Merged
merged 32 commits into from
Aug 1, 2023
Merged

Workflow update - tests, checks #60

merged 32 commits into from
Aug 1, 2023

Conversation

delihus
Copy link
Contributor

@delihus delihus commented Aug 1, 2023 •
edited
Loading

Included:

  • tests on PR,
  • tests on commit to the humble branch,
  • updating version,
  • trigger deploy docker image.

delihus and others added 30 commits July 14, 2023 15:30
@delihus
Create industrial_ci_action.yml
035b0e7
@delihus
Update industrial_ci_action.yml
96e2455
@delihus
Update README.md
0955427
@delihus
Update industrial_ci_action.yml
bbfcc24
@delihus
pre-requirements
4eb9050 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
9ec9a6c 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
f25468c 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
dcefa82 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
e017c1a 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
533e54d 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
5e7e771 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
082ffb4 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
fa6e90c 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
1f772bc 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
a387ff8 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
08b44f3 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
58fdb32 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
b881ea6 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
19ee331 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
986f5a9 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
ca1e5fb 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
51f5cba 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
Merge pull request #3 from delihus/delihus-patch-1
10861ac 
Update README.md
@delihus
pre-requirements
0331803 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
Merge remote-tracking branch 'origin/humble' into humble
59a22bb
@delihus
pre-requirements
284fe59 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
90e2676 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
6f8fe7d 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
1a595a9 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
ed7e960 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus
pre-requirements
8890780 
Signed-off-by: Jakub Delicat <[email protected]>
@delihus delihus requested a review from macstepien August 1, 2023 08:11
@delihus delihus closed this Aug 1, 2023
@delihus delihus reopened this Aug 1, 2023
macstepien
macstepien reviewed Aug 1, 2023
View reviewed changes
Copy link
Contributor

@macstepien macstepien left a comment

Choose a reason for hiding this comment

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

I'm not sure if check_pr_description_bump.yaml is necessary. In some cases (for example when updating workflows), I skip bumping. Instead we can use pull request template as in the rosbot_xl_ros repo.

@delihus delihus requested a review from macstepien August 1, 2023 11:31
@delihus delihus merged commit 51b029b into husarion:humble Aug 1, 2023
@delihus delihus mentioned this pull request Aug 1, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@macstepien macstepien macstepien 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.
2 participants
@delihus @macstepien