Skip to content

remove typhon and vendor dir#48

Closed
DaytonG wants to merge 1 commit intoredboxllc:masterfrom
DaytonG:feature/dgomez-remove-typhon
Closed

remove typhon and vendor dir#48
DaytonG wants to merge 1 commit intoredboxllc:masterfrom
DaytonG:feature/dgomez-remove-typhon

Conversation

@DaytonG
Copy link
Copy Markdown

@DaytonG DaytonG commented Jul 27, 2021

Ref: #47

Removed the tilt changes, though I think I'll keep that other PR open for now and just use it for the tilt proposal.

In this PR:

  • removed typhon (which drastically reduces security footprint)
  • removed vendor/ dir in favor of go modules

@DaytonG
Copy link
Copy Markdown
Author

DaytonG commented Aug 12, 2021

Ping on this, if there's anyone available to review.

@linjmeyer
Copy link
Copy Markdown

Hey @DaytonG I didn't forget about you - just on vacation at the moment. I'll take a look next week.

@linjmeyer linjmeyer self-assigned this Aug 12, 2021
@linjmeyer linjmeyer self-requested a review August 12, 2021 22:52
@DaytonG
Copy link
Copy Markdown
Author

DaytonG commented Sep 14, 2021

Thanks! Hopefully this can get a look soon.

jgielstra-cs added a commit to CognitiveScale/scuttle that referenced this pull request Apr 12, 2022
@linjmeyer
Copy link
Copy Markdown

Hello, I am leaving the Redbox org and will close this PR as I won't have time to review and it is several months old already

@linjmeyer linjmeyer closed this May 31, 2022
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.

2 participants