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.

Sign up for GitHub

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

Jump to bottom

Add update tools script #123

Merged
merged 1 commit into from
Aug 29, 2024
Merged

Add update tools script #123

merged 1 commit into from
Aug 29, 2024
+896 −138

Conversation

neekolas
Copy link
Contributor

@neekolas neekolas commented Aug 29, 2024 •
edited
Loading

tl;dr

  • Adds a script to force your tools to match the go.mod values
  • Moves some more tools to a Go install instead of Homebrew

This was referenced Aug 29, 2024
Merged
Merged
@neekolas Graphite App
Copy link
Contributor Author

neekolas commented Aug 29, 2024 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @neekolas and the rest of your teammates on Graphite Graphite

@neekolas neekolas mentioned this pull request Aug 29, 2024
Merged
@neekolas neekolas requested a review from richardhuaaa August 29, 2024 05:22
@neekolas neekolas marked this pull request as ready for review August 29, 2024 05:23
@neekolas neekolas force-pushed the 08-28-add_update_tools_script branch from 8d70a07 to f385794 Compare August 29, 2024 05:26
@neekolas neekolas force-pushed the 08-28-add_e2e_test_for_indexing_from_the_chain branch from 305811d to 6d0a129 Compare August 29, 2024 22:56
@neekolas neekolas force-pushed the 08-28-add_update_tools_script branch from f385794 to 332b8e9 Compare August 29, 2024 22:56
@neekolas Graphite App
Copy link
Contributor Author

neekolas commented Aug 29, 2024 •
edited
Loading

Merge activity

  • Aug 29, 4:23 PM PDT: @neekolas started a stack merge that includes this pull request via Graphite.
  • Aug 29, 4:33 PM PDT: Graphite rebased this pull request as part of a merge.
  • Aug 29, 4:35 PM PDT: @neekolas merged this pull request with Graphite.

@neekolas neekolas changed the base branch from 08-28-add_e2e_test_for_indexing_from_the_chain to graphite-base/123 August 29, 2024 23:28
@neekolas neekolas changed the base branch from graphite-base/123 to main August 29, 2024 23:31
@neekolas neekolas force-pushed the 08-28-add_update_tools_script branch from 332b8e9 to 327b0fe Compare August 29, 2024 23:32
@neekolas neekolas merged commit ac08512 into main Aug 29, 2024
5 checks passed
@neekolas neekolas deleted the 08-28-add_update_tools_script branch August 29, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mkysel mkysel mkysel approved these changes

@richardhuaaa richardhuaaa richardhuaaa approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@neekolas @richardhuaaa @mkysel