Skip to content
This repository has been archived by the owner on Oct 2, 2022. It is now read-only.

Update dependencies #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

GeoffreyFrogeye
Copy link
Contributor

This PR is split in 3 commits that:

  1. Updates the license year to 2021
  2. Update dependencies
  3. Removes the vendor/ directory from the repository

The last one I did because I don't want to clutter the history of the repository with stuff that can be retrieved from elsewhere. It is already quite heavy (103 MiB for 279 KiB of original code). But maybe you have a good reason that it's here in the first place.

I tested it and it seems to work fine. The Dockerfile seems to build fine too.

@s32x
Copy link
Owner

s32x commented Feb 17, 2021

Hey, Thanks for the PR. I prefer to keep vendor since go relies on code being checked into github. The second a dependency(repo) is deleted, this project is SOL. Until there's a central storage solution that cannot be removed, committing vendor is necessary.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants