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

fixes Issue 45 - cant generate using ent and then run #48

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

gedw99
Copy link
Contributor

@gedw99 gedw99 commented Nov 14, 2023

#45

Al I did was run go-mod-upgrade and go mod tidy
https://github.com/oligot/go-mod-upgrade

This does a sweep through everything. I updated to all latest dependencies, and did a generate and run and test.

This is my first PR, so if I missed something or should have done it on a branch, let me know.

@gedw99 gedw99 changed the title fixes 45 - cant generate fixes Issue 45 - cant generate using ent and then run Nov 14, 2023
@gedw99
Copy link
Contributor Author

gedw99 commented Nov 14, 2023

https://github.com/mikestefanello/pagoda/blob/main/pkg/context/context_test.go#L21 is the source of a test failure on my laptop. It's minor I think. Not sure if it was there before

@gedw99
Copy link
Contributor Author

gedw99 commented Nov 14, 2023

@mikestefanello do you require that PRs be signed ?

@mikestefanello
Copy link
Owner

Thanks for taking care of this. I'll fix the context_test issue now.

@mikestefanello mikestefanello merged commit ba96445 into mikestefanello:main Nov 16, 2023
1 check passed
@gedw99
Copy link
Contributor Author

gedw99 commented Nov 18, 2023

Thanks for taking care of this. I'll fix the context_test issue now.

thanks @mikestefanello

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.

None yet

2 participants