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

re-organize tool installation #127

Merged
merged 1 commit into from
Sep 15, 2023
Merged

re-organize tool installation #127

merged 1 commit into from
Sep 15, 2023

Commits on Sep 7, 2023

  1. re-organize tool installation

    Before this change, tools were installed on the user's system, outside of the
    project directory, some into `npm --global prefix`, others in `GOPATH`. This
    can be annoying, as it can overwrite existing tools, or different projects
    with requirements on different versions of tools can conflict.
    
    The tools have been re-organized, so that they're all installed by the
    Makefile, and placed within the project directory. Specifically, go binaries
    are in tools/bin, and node scripts are in node_modules/.bin. The scripts have
    been edited to modify their paths to prefer the use of these project-specified
    tools, so that conflicts with other user-installed tools should be prevented.
    
    I also spent a little time doing some shell script cleanup, little things like
    quoting arguments, checking for required arguments and the like.
    
    Oh, and I versioned the go tools that were being installed. No one likes
    having those @latest surprises on a production deployment!
    ewollesen committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    3b4c641 View commit details
    Browse the repository at this point in the history