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 go example #1439

Merged
merged 1 commit into from
Sep 20, 2024
Merged

add go example #1439

merged 1 commit into from
Sep 20, 2024

Conversation

sebastianwebber
Copy link
Contributor

@sebastianwebber sebastianwebber commented Sep 11, 2024

This PR adds an example to build go applications.

It adds:

  • the input for gomod2nix
  • the output with the example app

feel free to update anything needed or let me know what needs to be changed.

examples/go/.envrc Outdated Show resolved Hide resolved
@domenkozar
Copy link
Member

Can you run pre-commit run -a

@sebastianwebber
Copy link
Contributor Author

is this error expected?

✦2 ❯ devenv shell
• Building shell ...
• Using Cachix: devenv
warning: creating lock file '/Users/seba/projetos/github.com/sebastianwebber/devenv/examples/go/devenv.lock'
✔ Building shell in 6.7s.
• Entering shell
✨ devenv 1.1.0 is newer than devenv input in devenv.lock. Run `devenv update` to sync.
(devenv) bash-5.2$ pre-commit -a
usage: pre-commit [-h] [-V] {autoupdate,clean,gc,init-templatedir,install,install-hooks,migrate-config,run,sample-config,try-repo,uninstall,validate-config,validate-manifest,help,hook-impl} ...
pre-commit: error: unrecognized arguments: -a

without the -a, it works:

(devenv) bash-5.2$ pre-commit
[WARNING] Unstaged files detected.
[INFO] Stashing unstaged files to /Users/seba/.cache/pre-commit/patch1726164631-49152.
golangci-lint........................................(no files to check)Skipped
gotest...............................................(no files to check)Skipped
govet................................................(no files to check)Skipped
[INFO] Restored changes from /Users/seba/.cache/pre-commit/patch1726164631-49152.

same for devenv test:

(devenv) bash-5.2$ devenv test
• Overriding .devenv to .devenv.NYFQiooY7g6D
• Building tests ...
• Using Cachix: devenv
warning: creating lock file '/Users/seba/projetos/github.com/sebastianwebber/devenv/examples/go/devenv.lock'
✔ Building tests in 7.9s.
warning: creating lock file '/Users/seba/projetos/github.com/sebastianwebber/devenv/examples/go/devenv.lock'
• Running tests ...
• Building shell ...
warning: creating lock file '/Users/seba/projetos/github.com/sebastianwebber/devenv/examples/go/devenv.lock'
✔ Building shell in 3.4s.
✨ devenv 1.1.0 is newer than devenv input in devenv.lock. Run `devenv update` to sync.
• Setting up shell environment ...
✨ devenv 1.1.0 is newer than devenv input in devenv.lock. Run `devenv update` to sync.
• Testing ...
golangci-lint........................................(no files to check)Skipped
gotest...............................................(no files to check)Skipped
govet................................................(no files to check)Skipped
✔ Running tests in 6.1s.
• Tests passed :)

Signed-off-by: Sebastian Webber <sebastian@swebber.me>
@domenkozar
Copy link
Member

(devenv) bash-5.2$ pre-commit -a

You need to run pre-commit run -a.

@sebastianwebber
Copy link
Contributor Author

there you go:

(devenv) bash-5.2$ pre-commit run -a
golangci-lint........................................(no files to check)Skipped
gotest...............................................(no files to check)Skipped
govet................................................(no files to check)Skipped

@domenkozar
Copy link
Member

@sebastianwebber did you run that in examples/go env?

@sebastianwebber
Copy link
Contributor Author

hey @domenkozar , yes, i did:

pwd
/Users/seba/projetos/github.com/sebastianwebber/devenv/examples/go

❯ devenv shell
• Building shell ...
• Using Cachix: devenv
warning: creating lock file '/Users/seba/projetos/github.com/sebastianwebber/devenv/examples/go/devenv.lock'
✔ Building shell in 68.9s.
• Entering shell
(devenv) bash-5.2$ pre-commit run -a
golangci-lint........................................(no files to check)Skipped
gotest...............................................(no files to check)Skipped
govet................................................(no files to check)Skipped

Should I run the command in another folder?

please let me know how to proceed.

@domenkozar domenkozar merged commit 9513cce into cachix:main Sep 20, 2024
7 of 11 checks passed
@domenkozar
Copy link
Member

I got confused for a second as well, formatting had to be run at the top-level of the repo.

I've manually merged this, thank you!

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