Skip to content

Conversation

@technowhizz
Copy link

@technowhizz technowhizz commented Aug 3, 2023

Adds support for .deb packages as well as .zips

@technowhizz technowhizz requested a review from jovial August 3, 2023 15:46
@technowhizz technowhizz self-assigned this Aug 3, 2023
@technowhizz technowhizz requested a review from a team as a code owner August 3, 2023 15:46
@technowhizz technowhizz force-pushed the deb_support branch 5 times, most recently from 8eab588 to 31f1c1a Compare August 3, 2023 16:04
jovial
jovial previously approved these changes Aug 4, 2023
Copy link
Collaborator

@jovial jovial left a comment

Choose a reason for hiding this comment

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

LGTM, but please test before merging

jovial
jovial previously approved these changes Aug 4, 2023
@jovial
Copy link
Collaborator

jovial commented Aug 4, 2023

Probably should mention .deb support in the docs

@jovial jovial dismissed their stale review August 4, 2023 16:08

Need docs

@jovial
Copy link
Collaborator

jovial commented Mar 1, 2024

Lets get this in before it gets stale. Can you add docs in a follow up?

ansible.builtin.file:
path: "{{ install_script }}"
mode: u+x
- name: Intall a zip
Copy link
Collaborator

Choose a reason for hiding this comment

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

Typo

@technowhizz
Copy link
Author

Will fix soon

@markgoddard markgoddard requested a review from jovial September 10, 2024 08:35
@MoteHue
Copy link
Contributor

MoteHue commented Sep 25, 2024

@technowhizz what's the latest on this one?

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.

4 participants