keybase / saltpack

a modern crypto messaging format
https://saltpack.org/
BSD 3-Clause "New" or "Revised" License
989 stars 63 forks source link

Remove Saltpack prefixes from some consts and vars #20

Closed akalin-keybase closed 7 years ago

akalin-keybase commented 7 years ago

This is the recommended Go style.

Also convert some exported vars to functions to avoid modifications.

Fix a few typos and remove a few extraneous newlines.

Add a TODO for checking FormatName.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 86.215% when pulling 8feedbaae05f86e92bea178530707638a7a68d91 on akalin/fix-prefix into a5352c4421c37856db346939ff6d267af0bcaea3 on master.

akalin-keybase commented 7 years ago

This is pulled out of https://github.com/keybase/saltpack/pull/18 , and is first in a series.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.05%) to 86.269% when pulling 0b64831a0dcb48d909cd6b5c9c5c3a8f57fb1bf8 on akalin/fix-prefix into a5352c4421c37856db346939ff6d267af0bcaea3 on master.