search

datacequia / go-dogg3rz

Decentralized Knowledge Orchestration Tool
Apache License 2.0
8 stars 1 forks source link

add golang context package/object to dogg3rz command API #6

Closed adpadilla closed 3 years ago

adpadilla commented 3 years ago

Previously used environment variables in code directly which was problematic when creating unit tests because tests can run concurrently. Opted instead to have code that require environment variables to use context object instead where equivalent environment variable is set and is concurrency safe.

adpadilla commented 3 years ago

updates have been pushed. project should now build. Will still make a few more commits that take care of places where method signature that passes a context object is NOT first argument (this is standard convention)

adpadilla commented 3 years ago

done:

Andrews-MacBook-Pro:go-dogg3rz andrew$ git push Enumerating objects: 43, done. Counting objects: 100% (43/43), done. Delta compression using up to 12 threads Compressing objects: 100% (20/20), done. Writing objects: 100% (22/22), 2.45 KiB | 2.45 MiB/s, done. Total 22 (delta 17), reused 0 (delta 0), pack-reused 0 remote: Resolving deltas: 100% (17/17), completed with 17 local objects. To https://github.com/datacequia/go-dogg3rz.git 06aee7e..dd7d750 master -> master