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

Add support for GitHub codespaces and minor cleanup #179

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

NiceGuyIT
Copy link
Member

@NiceGuyIT NiceGuyIT commented Nov 22, 2023

Chg: Add support for GitHub codespaces (devcontainers)
Chg: Move scripts to scripts/ directory

Updated:

GitHub Codespaces provides a dev environment (VM) in the cloud. The goal is to be able to easily test PRs without doing a git clone. To accomplish that, a Codespace can be started, the smartctl_exporter is compiled and run, and a port is exposed to view the data. This can be done on master in this repo, and then checking out the PR commit. (I don't think Codespaces can be started from a PR commit.) Once done, the Codespace can be deleted. watchexec and just/task are cross platform which will help when working on Windows support (issue #163). watchexec and just/task will be in a separate PR.

  1. Add support for watchexec to build the exporter upon changes.
  2. Use a task runner such as just or task to ease the build/run/test process.

Signed-off-by: David Randall <[email protected]>
Signed-off-by: David Randall <[email protected]>
Chg: Add support for GitHub codespaces (devcontainers)
Chg: Minor cleanup

Signed-off-by: David Randall <[email protected]>
Signed-off-by: David Randall <[email protected]>
@SuperQ
Copy link
Contributor

SuperQ commented Nov 22, 2023

What are the codespaces for? Can you add a bit of a "why" to the PR description?

@NiceGuyIT
Copy link
Member Author

What are the codespaces for? Can you add a bit of a "why" to the PR description?

Done. Thank you for the reminder. I should have explained in the PR.

@SuperQ
Copy link
Contributor

SuperQ commented Nov 25, 2023

Hmm, interesting idea. I think something like this should be discussed with the wider Prometheus dev community. It would be better to support this kind of thing consistently across all projects.

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.

3 participants