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

Generate DET executable files #232

Closed
wants to merge 6 commits into from
Closed

Generate DET executable files #232

wants to merge 6 commits into from

Conversation

Charl1996
Copy link
Contributor

@Charl1996 Charl1996 commented Feb 5, 2024

Ticket

This PR broadens the DET horizons by introducing functionality to generate/compile executable files of the DET, each compatible with a different operating system. The supported operating systems are

  1. Linux
  2. Windows

Note that these two executable files are compiled automatically on every release publish and will be added as assets to the release once the workflow completes. Any person can then simply download the file(s) from there and use it (through the CLI) like you would as if you installed commcare-export,

commcare-export < options >

The big push for this was the fact that some folks using the DET is less technically-inclined and having to manage a python codebase and it's packages poses a barrier to usage. Now users can simply download and run the file without thinking about any installation steps.

- name: Upload release assets
uses: AButler/upload-release-assets@v3.0
with:
files: "./dist/linux/*;./dist/windows/*"
Copy link
Contributor

Choose a reason for hiding this comment

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

where would we find this? Or does it automatically attach the asset to the release?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, the files are automatically attached to the release as assets.

@Charl1996 Charl1996 marked this pull request as draft February 6, 2024 17:23
@Charl1996
Copy link
Contributor Author

Charl1996 commented Feb 6, 2024

Putting this back into draft as I have discovered something that makes this a non-working solution.

@Charl1996
Copy link
Contributor Author

Will open new (working) PR

@Charl1996 Charl1996 closed this Mar 7, 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