[go: nahoru, domu]

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

Run without issues on redhat systems and update drawio. #1

Merged
merged 4 commits into from
May 22, 2021
Merged

Run without issues on redhat systems and update drawio. #1

merged 4 commits into from
May 22, 2021

Conversation

Mosibi
Copy link
@Mosibi Mosibi commented May 21, 2021

No description provided.

Richard Arends added 4 commits May 21, 2021 13:40
…e volume.

Detect if podman or docker is present. On recent Red Hat systems, the podman
configuration is by default rootless, therefor prefer podman.
Copy link
Contributor
@CoMPaTech CoMPaTech left a comment

Choose a reason for hiding this comment

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

Looking good, I'll write up some flows as well to quickly test functionality

@CoMPaTech CoMPaTech mentioned this pull request May 22, 2021
@CoMPaTech
Copy link
Contributor

@Mosibi Actions will bork if we merge I presume (because the action still runs the old shell-script and requirements moved on) - we can just merge (as apparently you can't rerun newer actions on existing PRs) and fix with another PR or fix it in the PR. Both are fine with me :)

@Mosibi
Copy link
Author
Mosibi commented May 22, 2021

@Mosibi Actions will bork if we merge I presume (because the action still runs the old shell-script and requirements moved on) - we can just merge (as apparently you can't rerun newer actions on existing PRs) and fix with another PR or fix it in the PR. Both are fine with me :)

Let’s just merge it and fix any issues later, if they occur

@CoMPaTech
Copy link
Contributor

👍

@CoMPaTech CoMPaTech merged commit 21e7dc3 into suecodelabs:main May 22, 2021
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.

None yet

2 participants