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

dep init #77

Closed
wants to merge 1 commit into from
Closed

dep init #77

wants to merge 1 commit into from

Conversation

sudo-suhas
Copy link

What does this do?

Closes #72.

@sudo-suhas
Copy link
Author

@matthewmueller How do you configure semaphoreci? It would be better if CI also used dep for getting the dependencies. Here's some relevant documentation for using dep in CI - golang/dep:docs/FAQ.md@master#how-do-i-use-dep-in-ci.

@matthewmueller
Copy link
Owner

hey thanks for this! did you do anything special to get dep init working? i had troubles last time i tried.

definitely want to get dep going. agreed on the CI integration. i think we can just add a makefile command to semaphore to add this step.

please give me a couple days to test this out!

@sudo-suhas
Copy link
Author

did you do anything special to get dep init working?

No, I did not have to do anything special. I installed dep using go get. Maybe that makes the difference?

i think we can just add a makefile command to semaphore

Should I push a commit which updates the install step in the makefile?

please give me a couple days to test this out!

Sure.

@sudo-suhas
Copy link
Author

ping @matthewmueller.

@sudo-suhas sudo-suhas closed this Nov 25, 2022
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.

Using a package/dependency manager
2 participants