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 support for Go workspaces #999

Merged

Commits on Jul 4, 2024

  1. Add support for Go workspaces

    Go 1.18 introduced support for workspaces¹, which were until now
    explicitly forbidden by Cachito when processing a request.
    
    The main change this feature brings is the return value for the "go list
    -m" command, which was used to retrieve the name of each of the modules
    specified as input packages in a Cachito request.
    
    This command now returns a list of all workspaces defined in the
    "go.work" file, which we will use to properly list them as part of the
    request's output.
    
    ¹ https://go.dev/ref/mod#workspaces
    
    Signed-off-by: Bruno Pimentel <[email protected]>
    brunoapimentel committed Jul 4, 2024
    Configuration menu
    Copy the full SHA
    deeedc1 View commit details
    Browse the repository at this point in the history
  2. Remove the restriction in using Go workspaces

    Since the support for workspaces was not properly implemented, we had a
    strict check on the existence of a "go.work" file in a repository, which
    would then trigger a failure in a request.
    
    Now that proper support is implemented, let's remove this check.
    
    Signed-off-by: Bruno Pimentel <[email protected]>
    brunoapimentel committed Jul 4, 2024
    Configuration menu
    Copy the full SHA
    99869a2 View commit details
    Browse the repository at this point in the history
  3. Add integration test to cover Go workspaces

    Signed-off-by: Bruno Pimentel <[email protected]>
    brunoapimentel committed Jul 4, 2024
    Configuration menu
    Copy the full SHA
    88b8e75 View commit details
    Browse the repository at this point in the history