Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

TLS functions in go/db/db.go to go/db/tls.go #155

Closed
wants to merge 3 commits into from

Conversation

sjmudd
Copy link
Collaborator

@sjmudd sjmudd commented Apr 19, 2017

This is patch 2 of #139. which moves some of the files out of db.go to a separate file. No functional change, this just moves things around a bit.

  • contributed code is using same conventions as original code
  • code is formatted via gofmt (please avoid goimports)
  • code is built via ./build.sh
  • code is tested via go test ./go/... # well not really. They fail when run against the github master branch (TLS tests fail). Maybe we should figure out how to make them pass or my environment is not as expected. Does this work for you? $ cd go/db && go test does work for me.

@shlomi-noach
Copy link
Collaborator

closing based on @sjmudd 's comment: #242 (comment)

@sjmudd sjmudd deleted the tls_move_functions branch December 25, 2021 20:17
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