Preparing report...

Report for github.com/kinecosystem/go

A+    Excellent!    Found 160 issues across 456 files

Tweet

gofmt90%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command


go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!


golint75%

Golint is a linter for Go source code.

    • go/support/render/hal/link.go
    • Line 7: warning: exported type Link should have comment or be unexported (golint)
    • Line 12: warning: exported method Link.PopulateTemplated should have comment or be unexported (golint)
    • Line 16: warning: exported function NewLink should have comment or be unexported (golint)
    • go/clients/horizon/main.go
    • Line 52: warning: exported const OrderAsc should have comment (or a comment on this block) or be unexported (golint)
    • Line 86: warning: exported type ClientInterface should have comment or be unexported (golint)
    • go/services/horizon/internal/test/http.go
    • Line 12: warning: exported type RequestHelper should have comment or be unexported (golint)
    • Line 21: warning: exported function RequestHelperRemoteAddr should have comment or be unexported (golint)
    • Line 27: warning: exported function RequestHelperXFF should have comment or be unexported (golint)
    • Line 33: warning: exported function RequestHelperRaw should have comment or be unexported (golint)
    • Line 37: warning: exported function RequestHelperStreaming should have comment or be unexported (golint)
    • Line 41: warning: exported function NewRequestHelper should have comment or be unexported (golint)
    • go/services/horizon/internal/context/context_key.go
    • Line 3: warning: exported type CtxKey should have comment or be unexported (golint)
    • Line 5: warning: exported var AppContextKey should have comment or be unexported (golint)
    • Line 6: warning: exported var RequestContextKey should have comment or be unexported (golint)
    • Line 7: warning: exported var ClientContextKey should have comment or be unexported (golint)
    • go/services/horizon/internal/init_web.go
    • Line 189: warning: exported type VaryByRemoteIP should have comment or be unexported (golint)
    • Line 191: warning: exported method VaryByRemoteIP.Key should have comment or be unexported (golint)
    • Line 200: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • go/tools/stellar-archivist/internal/util.go
    • Line 54: warning: exported function Categories should have comment or be unexported (golint)
    • Line 61: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 70: warning: exported function CategoryCheckpointPath should have comment or be unexported (golint)
    • Line 76: warning: exported function BucketPath should have comment or be unexported (golint)
    • go/meta/bundle.go
    • Line 107: warning: comment on exported method Bundle.OperationsMetas should be of the form "OperationsMetas ..." (golint)
    • go/tools/stellar-archivist/internal/fs_archive.go
    • Line 14: warning: exported type FsArchiveBackend should have comment or be unexported (golint)
    • Line 18: warning: exported method FsArchiveBackend.GetFile should have comment or be unexported (golint)
    • Line 22: warning: exported method FsArchiveBackend.Exists should have comment or be unexported (golint)
    • Line 31: warning: exported method FsArchiveBackend.PutFile should have comment or be unexported (golint)
    • Line 50: warning: exported method FsArchiveBackend.ListFiles should have comment or be unexported (golint)
    • Line 71: warning: exported method FsArchiveBackend.CanListFiles should have comment or be unexported (golint)
    • Line 75: warning: exported function MakeFsBackend should have comment or be unexported (golint)
    • go/clients/horizon/responses.go
    • Line 9: warning: comment on exported type Problem should be of the form "Problem ..." (with optional leading article) (golint)
    • Line 19: warning: comment on exported type Root should be of the form "Root ..." (with optional leading article) (golint)
    • Line 22: warning: comment on exported type Account should be of the form "Account ..." (with optional leading article) (golint)
    • Line 25: warning: comment on exported type AccountFlags should be of the form "AccountFlags ..." (with optional leading article) (golint)
    • Line 28: warning: comment on exported type AccountThresholds should be of the form "AccountThresholds ..." (with optional leading article) (golint)
    • Line 31: warning: comment on exported type Asset should be of the form "Asset ..." (with optional leading article) (golint)
    • Line 34: warning: comment on exported type Balance should be of the form "Balance ..." (with optional leading article) (golint)
    • Line 37: warning: comment on exported type HistoryAccount should be of the form "HistoryAccount ..." (with optional leading article) (golint)
    • Line 40: warning: comment on exported type Ledger should be of the form "Ledger ..." (with optional leading article) (golint)
    • Line 43: warning: comment on exported type Link should be of the form "Link ..." (with optional leading article) (golint)
    • Line 46: warning: comment on exported type Offer should be of the form "Offer ..." (with optional leading article) (golint)
    • Line 49: warning: comment on exported type EffectsPage should be of the form "EffectsPage ..." (with optional leading article) (golint)
    • Line 57: warning: comment on exported type Effect should be of the form "Effect ..." (with optional leading article) (golint)
    • Line 72: warning: comment on exported type TradeAggregation should be of the form "TradeAggregation ..." (with optional leading article) (golint)
    • Line 83: warning: comment on exported type Trade should be of the form "Trade ..." (with optional leading article) (golint)
    • Line 86: warning: comment on exported type OrderBookSummary should be of the form "OrderBookSummary ..." (with optional leading article) (golint)
    • Line 89: warning: comment on exported type TransactionSuccess should be of the form "TransactionSuccess ..." (with optional leading article) (golint)
    • Line 92: warning: comment on exported type TransactionResultCodes should be of the form "TransactionResultCodes ..." (with optional leading article) (golint)
    • Line 95: warning: comment on exported type Signer should be of the form "Signer ..." (with optional leading article) (golint)
    • Line 106: warning: exported type Payment should have comment or be unexported (golint)
    • Line 149: warning: comment on exported type Price should be of the form "Price ..." (with optional leading article) (golint)
    • Line 152: warning: comment on exported type PriceLevel should be of the form "PriceLevel ..." (with optional leading article) (golint)
    • Line 155: warning: comment on exported type Transaction should be of the form "Transaction ..." (with optional leading article) (golint)
    • go/xdr/xdr_generated.go
    • Line 102: warning: exported const ScpStatementTypeScpStPrepare should have comment (or a comment on this block) or be unexported (golint)
    • Line 129: warning: receiver name s should be consistent with previous receiver name e for ScpStatementType (golint)
    • Line 136: warning: receiver name s should be consistent with previous receiver name e for ScpStatementType (golint)
    • Line 489: warning: receiver name s should be consistent with previous receiver name u for ScpStatementPledges (golint)
    • Line 496: warning: receiver name s should be consistent with previous receiver name u for ScpStatementPledges (golint)
    • Line 673: warning: receiver name s should be consistent with previous receiver name u for AccountId (golint)
    • Line 680: warning: receiver name s should be consistent with previous receiver name u for AccountId (golint)
    • Line 702: warning: receiver name s should be consistent with previous receiver name e for Thresholds (golint)
    • Line 709: warning: receiver name s should be consistent with previous receiver name e for Thresholds (golint)
    • Line 731: warning: receiver name s should be consistent with previous receiver name e for String32 (golint)
    • Line 738: warning: receiver name s should be consistent with previous receiver name e for String32 (golint)
    • Line 760: warning: receiver name s should be consistent with previous receiver name e for String64 (golint)
    • Line 767: warning: receiver name s should be consistent with previous receiver name e for String64 (golint)
    • Line 813: warning: receiver name s should be consistent with previous receiver name e for DataValue (golint)
    • Line 820: warning: receiver name s should be consistent with previous receiver name e for DataValue (golint)
    • Line 842: warning: exported const AssetTypeAssetTypeNative should have comment (or a comment on this block) or be unexported (golint)
    • Line 867: warning: receiver name s should be consistent with previous receiver name e for AssetType (golint)
    • Line 874: warning: receiver name s should be consistent with previous receiver name e for AssetType (golint)
    • Line 1071: warning: receiver name s should be consistent with previous receiver name u for Asset (golint)
    • Line 1078: warning: receiver name s should be consistent with previous receiver name u for Asset (golint)
    • Line 1163: warning: exported const ThresholdIndexesThresholdMasterWeight should have comment (or a comment on this block) or be unexported (golint)
    • Line 1190: warning: receiver name s should be consistent with previous receiver name e for ThresholdIndexes (golint)
    • Line 1197: warning: receiver name s should be consistent with previous receiver name e for ThresholdIndexes (golint)
    • Line 1220: warning: exported const LedgerEntryTypeAccount should have comment (or a comment on this block) or be unexported (golint)
    • Line 1247: warning: receiver name s should be consistent with previous receiver name e for LedgerEntryType (golint)
    • Line 1254: warning: receiver name s should be consistent with previous receiver name e for LedgerEntryType (golint)
    • Line 1314: warning: exported const AccountFlagsAuthRequiredFlag should have comment (or a comment on this block) or be unexported (golint)
    • Line 1339: warning: receiver name s should be consistent with previous receiver name e for AccountFlags (golint)
    • Line 1346: warning: receiver name s should be consistent with previous receiver name e for AccountFlags (golint)
    • Line 1401: warning: receiver name s should be consistent with previous receiver name u for AccountEntryV1Ext (golint)
    • Line 1408: warning: receiver name s should be consistent with previous receiver name u for AccountEntryV1Ext (golint)
    • Line 1541: warning: receiver name s should be consistent with previous receiver name u for AccountEntryExt (golint)
    • Line 1548: warning: receiver name s should be consistent with previous receiver name u for AccountEntryExt (golint)
    • Line 1641: warning: exported const TrustLineFlagsAuthorizedFlag should have comment (or a comment on this block) or be unexported (golint)
    • Line 1662: warning: receiver name s should be consistent with previous receiver name e for TrustLineFlags (golint)
    • Line 1669: warning: receiver name s should be consistent with previous receiver name e for TrustLineFlags (golint)
    • Line 1724: warning: receiver name s should be consistent with previous receiver name u for TrustLineEntryV1Ext (golint)
    • Line 1731: warning: receiver name s should be consistent with previous receiver name u for TrustLineEntryV1Ext (golint)
    • Line 1864: warning: receiver name s should be consistent with previous receiver name u for TrustLineEntryExt (golint)
    • Line 1871: warning: receiver name s should be consistent with previous receiver name u for TrustLineEntryExt (golint)
    • Line 1952: warning: exported const OfferEntryFlagsPassiveFlag should have comment (or a comment on this block) or be unexported (golint)
    • Line 1973: warning: receiver name s should be consistent with previous receiver name e for OfferEntryFlags (golint)
    • Line 1980: warning: receiver name s should be consistent with previous receiver name e for OfferEntryFlags (golint)
    • Line 2035: warning: receiver name s should be consistent with previous receiver name u for OfferEntryExt (golint)
    • Line 2042: warning: receiver name s should be consistent with previous receiver name u for OfferEntryExt (golint)
    • Line 2147: warning: receiver name s should be consistent with previous receiver name u for DataEntryExt (golint)
    • Line 2154: warning: receiver name s should be consistent with previous receiver name u for DataEntryExt (golint)
    • Line 2387: warning: receiver name s should be consistent with previous receiver name u for LedgerEntryData (golint)
    • Line 2394: warning: receiver name s should be consistent with previous receiver name u for LedgerEntryData (golint)
    • Line 2443: warning: receiver name s should be consistent with previous receiver name u for LedgerEntryExt (golint)
    • Line 2450: warning: receiver name s should be consistent with previous receiver name u for LedgerEntryExt (golint)
    • Line 2524: warning: exported const EnvelopeTypeEnvelopeTypeScp should have comment (or a comment on this block) or be unexported (golint)
    • Line 2549: warning: receiver name s should be consistent with previous receiver name e for EnvelopeType (golint)
    • Line 2556: warning: receiver name s should be consistent with previous receiver name e for EnvelopeType (golint)
    • Line 2578: warning: receiver name s should be consistent with previous receiver name e for UpgradeType (golint)
    • Line 2585: warning: receiver name s should be consistent with previous receiver name e for UpgradeType (golint)
    • Line 2634: warning: receiver name s should be consistent with previous receiver name u for StellarValueExt (golint)
    • Line 2641: warning: receiver name s should be consistent with previous receiver name u for StellarValueExt (golint)
    • Line 2738: warning: receiver name s should be consistent with previous receiver name u for LedgerHeaderExt (golint)
    • Line 2745: warning: receiver name s should be consistent with previous receiver name u for LedgerHeaderExt (golint)
    • Line 2844: warning: exported const LedgerUpgradeTypeLedgerUpgradeVersion should have comment (or a comment on this block) or be unexported (golint)
    • Line 2871: warning: receiver name s should be consistent with previous receiver name e for LedgerUpgradeType (golint)
    • Line 2878: warning: receiver name s should be consistent with previous receiver name e for LedgerUpgradeType (golint)
    • Line 3069: warning: receiver name s should be consistent with previous receiver name u for LedgerUpgrade (golint)
    • Line 3076: warning: receiver name s should be consistent with previous receiver name u for LedgerUpgrade (golint)
    • Line 3407: warning: receiver name s should be consistent with previous receiver name u for LedgerKey (golint)
    • Line 3414: warning: receiver name s should be consistent with previous receiver name u for LedgerKey (golint)
    • Line 3435: warning: exported const BucketEntryTypeLiveentry should have comment (or a comment on this block) or be unexported (golint)
    • Line 3458: warning: receiver name s should be consistent with previous receiver name e for BucketEntryType (golint)
    • Line 3465: warning: receiver name s should be consistent with previous receiver name e for BucketEntryType (golint)
    • Line 3583: warning: receiver name s should be consistent with previous receiver name u for BucketEntry (golint)
    • Line 3590: warning: receiver name s should be consistent with previous receiver name u for BucketEntry (golint)
    • Line 3730: warning: receiver name s should be consistent with previous receiver name u for TransactionHistoryEntryExt (golint)
    • Line 3737: warning: receiver name s should be consistent with previous receiver name u for TransactionHistoryEntryExt (golint)
    • Line 3826: warning: receiver name s should be consistent with previous receiver name u for TransactionHistoryResultEntryExt (golint)
    • Line 3833: warning: receiver name s should be consistent with previous receiver name u for TransactionHistoryResultEntryExt (golint)
    • Line 3922: warning: receiver name s should be consistent with previous receiver name u for LedgerHeaderHistoryEntryExt (golint)
    • Line 3929: warning: receiver name s should be consistent with previous receiver name u for LedgerHeaderHistoryEntryExt (golint)
    • Line 4111: warning: receiver name s should be consistent with previous receiver name u for ScpHistoryEntry (golint)
    • Line 4118: warning: receiver name s should be consistent with previous receiver name u for ScpHistoryEntry (golint)
    • Line 4141: warning: exported const LedgerEntryChangeTypeLedgerEntryCreated should have comment (or a comment on this block) or be unexported (golint)
    • Line 4168: warning: receiver name s should be consistent with previous receiver name e for LedgerEntryChangeType (golint)
    • Line 4175: warning: receiver name s should be consistent with previous receiver name e for LedgerEntryChangeType (golint)
    • Line 4366: warning: receiver name s should be consistent with previous receiver name u for LedgerEntryChange (golint)
    • Line 4373: warning: receiver name s should be consistent with previous receiver name u for LedgerEntryChange (golint)
    • Line 4574: warning: receiver name s should be consistent with previous receiver name u for TransactionMeta (golint)
    • Line 4581: warning: receiver name s should be consistent with previous receiver name u for TransactionMeta (golint)
    • Line 4605: warning: exported const ErrorCodeErrMisc should have comment (or a comment on this block) or be unexported (golint)
    • Line 4634: warning: receiver name s should be consistent with previous receiver name e for ErrorCode (golint)
    • Line 4641: warning: receiver name s should be consistent with previous receiver name e for ErrorCode (golint)
    • Line 4802: warning: exported const IpAddrTypeIPv4 should have comment (or a comment on this block) or be unexported (golint)
    • Line 4825: warning: receiver name s should be consistent with previous receiver name e for IpAddrType (golint)
    • Line 4832: warning: receiver name s should be consistent with previous receiver name e for IpAddrType (golint)
    • Line 4949: warning: receiver name s should be consistent with previous receiver name u for PeerAddressIp (golint)
    • Line 4956: warning: receiver name s should be consistent with previous receiver name u for PeerAddressIp (golint)
    • Line 5035: warning: exported const MessageTypeErrorMsg should have comment (or a comment on this block) or be unexported (golint)
    • Line 5080: warning: receiver name s should be consistent with previous receiver name e for MessageType (golint)
    • Line 5087: warning: receiver name s should be consistent with previous receiver name e for MessageType (golint)
    • Line 5615: warning: receiver name s should be consistent with previous receiver name u for StellarMessage (golint)
    • Line 5622: warning: receiver name s should be consistent with previous receiver name u for StellarMessage (golint)
    • Line 5740: warning: receiver name s should be consistent with previous receiver name u for AuthenticatedMessage (golint)
    • Line 5747: warning: receiver name s should be consistent with previous receiver name u for AuthenticatedMessage (golint)
    • Line 5809: warning: exported const OperationTypeCreateAccount should have comment (or a comment on this block) or be unexported (golint)
    • Line 5852: warning: receiver name s should be consistent with previous receiver name e for OperationType (golint)
    • Line 5859: warning: receiver name s should be consistent with previous receiver name e for OperationType (golint)
    • Line 6246: warning: receiver name s should be consistent with previous receiver name u for AllowTrustOpAsset (golint)
    • Line 6253: warning: receiver name s should be consistent with previous receiver name u for AllowTrustOpAsset (golint)
    • Line 6814: warning: receiver name s should be consistent with previous receiver name u for OperationBody (golint)
    • Line 6821: warning: receiver name s should be consistent with previous receiver name u for OperationBody (golint)
    • Line 6907: warning: exported const MemoTypeMemoNone should have comment (or a comment on this block) or be unexported (golint)
    • Line 6936: warning: receiver name s should be consistent with previous receiver name e for MemoType (golint)
    • Line 6943: warning: receiver name s should be consistent with previous receiver name e for MemoType (golint)
    • Line 7140: warning: receiver name s should be consistent with previous receiver name u for Memo (golint)
    • Line 7147: warning: receiver name s should be consistent with previous receiver name u for Memo (golint)
    • Line 7227: warning: receiver name s should be consistent with previous receiver name u for TransactionExt (golint)
    • Line 7234: warning: receiver name s should be consistent with previous receiver name u for TransactionExt (golint)
    • Line 7372: warning: receiver name s should be consistent with previous receiver name u for TransactionSignaturePayloadTaggedTransaction (golint)
    • Line 7379: warning: receiver name s should be consistent with previous receiver name u for TransactionSignaturePayloadTaggedTransaction (golint)
    • Line 7521: warning: exported const CreateAccountResultCodeCreateAccountSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 7550: warning: receiver name s should be consistent with previous receiver name e for CreateAccountResultCode (golint)
    • Line 7557: warning: receiver name s should be consistent with previous receiver name e for CreateAccountResultCode (golint)
    • Line 7611: warning: receiver name s should be consistent with previous receiver name u for CreateAccountResult (golint)
    • Line 7618: warning: receiver name s should be consistent with previous receiver name u for CreateAccountResult (golint)
    • Line 7650: warning: exported const PaymentResultCodePaymentSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 7689: warning: receiver name s should be consistent with previous receiver name e for PaymentResultCode (golint)
    • Line 7696: warning: receiver name s should be consistent with previous receiver name e for PaymentResultCode (golint)
    • Line 7750: warning: receiver name s should be consistent with previous receiver name u for PaymentResult (golint)
    • Line 7757: warning: receiver name s should be consistent with previous receiver name u for PaymentResult (golint)
    • Line 7792: warning: exported const PathPaymentResultCodePathPaymentSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 7837: warning: receiver name s should be consistent with previous receiver name e for PathPaymentResultCode (golint)
    • Line 7844: warning: receiver name s should be consistent with previous receiver name e for PathPaymentResultCode (golint)
    • Line 8034: warning: receiver name s should be consistent with previous receiver name u for PathPaymentResult (golint)
    • Line 8041: warning: receiver name s should be consistent with previous receiver name u for PathPaymentResult (golint)
    • Line 8079: warning: exported const ManageOfferResultCodeManageOfferSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 8124: warning: receiver name s should be consistent with previous receiver name e for ManageOfferResultCode (golint)
    • Line 8131: warning: receiver name s should be consistent with previous receiver name e for ManageOfferResultCode (golint)
    • Line 8153: warning: exported const ManageOfferEffectManageOfferCreated should have comment (or a comment on this block) or be unexported (golint)
    • Line 8178: warning: receiver name s should be consistent with previous receiver name e for ManageOfferEffect (golint)
    • Line 8185: warning: receiver name s should be consistent with previous receiver name e for ManageOfferEffect (golint)
    • Line 8280: warning: receiver name s should be consistent with previous receiver name u for ManageOfferSuccessResultOffer (golint)
    • Line 8287: warning: receiver name s should be consistent with previous receiver name u for ManageOfferSuccessResultOffer (golint)
    • Line 8413: warning: receiver name s should be consistent with previous receiver name u for ManageOfferResult (golint)
    • Line 8420: warning: receiver name s should be consistent with previous receiver name u for ManageOfferResult (golint)
    • Line 8451: warning: exported const SetOptionsResultCodeSetOptionsSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 8490: warning: receiver name s should be consistent with previous receiver name e for SetOptionsResultCode (golint)
    • Line 8497: warning: receiver name s should be consistent with previous receiver name e for SetOptionsResultCode (golint)
    • Line 8551: warning: receiver name s should be consistent with previous receiver name u for SetOptionsResult (golint)
    • Line 8558: warning: receiver name s should be consistent with previous receiver name u for SetOptionsResult (golint)
    • Line 8587: warning: exported const ChangeTrustResultCodeChangeTrustSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 8618: warning: receiver name s should be consistent with previous receiver name e for ChangeTrustResultCode (golint)
    • Line 8625: warning: receiver name s should be consistent with previous receiver name e for ChangeTrustResultCode (golint)
    • Line 8679: warning: receiver name s should be consistent with previous receiver name u for ChangeTrustResult (golint)
    • Line 8686: warning: receiver name s should be consistent with previous receiver name u for ChangeTrustResult (golint)
    • Line 8714: warning: exported const AllowTrustResultCodeAllowTrustSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 8745: warning: receiver name s should be consistent with previous receiver name e for AllowTrustResultCode (golint)
    • Line 8752: warning: receiver name s should be consistent with previous receiver name e for AllowTrustResultCode (golint)
    • Line 8806: warning: receiver name s should be consistent with previous receiver name u for AllowTrustResult (golint)
    • Line 8813: warning: receiver name s should be consistent with previous receiver name u for AllowTrustResult (golint)
    • Line 8842: warning: exported const AccountMergeResultCodeAccountMergeSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 8875: warning: receiver name s should be consistent with previous receiver name e for AccountMergeResultCode (golint)
    • Line 8882: warning: receiver name s should be consistent with previous receiver name e for AccountMergeResultCode (golint)
    • Line 8967: warning: receiver name s should be consistent with previous receiver name u for AccountMergeResult (golint)
    • Line 8974: warning: receiver name s should be consistent with previous receiver name u for AccountMergeResult (golint)
    • Line 8997: warning: exported const InflationResultCodeInflationSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 9020: warning: receiver name s should be consistent with previous receiver name e for InflationResultCode (golint)
    • Line 9027: warning: receiver name s should be consistent with previous receiver name e for InflationResultCode (golint)
    • Line 9143: warning: receiver name s should be consistent with previous receiver name u for InflationResult (golint)
    • Line 9150: warning: receiver name s should be consistent with previous receiver name u for InflationResult (golint)
    • Line 9178: warning: exported const ManageDataResultCodeManageDataSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 9207: warning: receiver name s should be consistent with previous receiver name e for ManageDataResultCode (golint)
    • Line 9214: warning: receiver name s should be consistent with previous receiver name e for ManageDataResultCode (golint)
    • Line 9268: warning: receiver name s should be consistent with previous receiver name u for ManageDataResult (golint)
    • Line 9275: warning: receiver name s should be consistent with previous receiver name u for ManageDataResult (golint)
    • Line 9298: warning: exported const BumpSequenceResultCodeBumpSequenceSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 9321: warning: receiver name s should be consistent with previous receiver name e for BumpSequenceResultCode (golint)
    • Line 9328: warning: receiver name s should be consistent with previous receiver name e for BumpSequenceResultCode (golint)
    • Line 9382: warning: receiver name s should be consistent with previous receiver name u for BumpSequenceResult (golint)
    • Line 9389: warning: receiver name s should be consistent with previous receiver name u for BumpSequenceResult (golint)
    • Line 9413: warning: exported const OperationResultCodeOpInner should have comment (or a comment on this block) or be unexported (golint)
    • Line 9440: warning: receiver name s should be consistent with previous receiver name e for OperationResultCode (golint)
    • Line 9447: warning: receiver name s should be consistent with previous receiver name e for OperationResultCode (golint)
    • Line 9934: warning: receiver name s should be consistent with previous receiver name u for OperationResultTr (golint)
    • Line 9941: warning: receiver name s should be consistent with previous receiver name u for OperationResultTr (golint)
    • Line 10053: warning: receiver name s should be consistent with previous receiver name u for OperationResult (golint)
    • Line 10060: warning: receiver name s should be consistent with previous receiver name u for OperationResult (golint)
    • Line 10094: warning: exported const TransactionResultCodeTxSuccess should have comment (or a comment on this block) or be unexported (golint)
    • Line 10137: warning: receiver name s should be consistent with previous receiver name e for TransactionResultCode (golint)
    • Line 10144: warning: receiver name s should be consistent with previous receiver name e for TransactionResultCode (golint)
    • Line 10239: warning: receiver name s should be consistent with previous receiver name u for TransactionResultResult (golint)
    • Line 10246: warning: receiver name s should be consistent with previous receiver name u for TransactionResultResult (golint)
    • Line 10295: warning: receiver name s should be consistent with previous receiver name u for TransactionResultExt (golint)
    • Line 10302: warning: receiver name s should be consistent with previous receiver name u for TransactionResultExt (golint)
    • Line 10373: warning: receiver name s should be consistent with previous receiver name e for Hash (golint)
    • Line 10380: warning: receiver name s should be consistent with previous receiver name e for Hash (golint)
    • Line 10402: warning: receiver name s should be consistent with previous receiver name e for Uint256 (golint)
    • Line 10409: warning: receiver name s should be consistent with previous receiver name e for Uint256 (golint)
    • Line 10527: warning: exported const CryptoKeyTypeKeyTypeEd25519 should have comment (or a comment on this block) or be unexported (golint)
    • Line 10552: warning: receiver name s should be consistent with previous receiver name e for CryptoKeyType (golint)
    • Line 10559: warning: receiver name s should be consistent with previous receiver name e for CryptoKeyType (golint)
    • Line 10579: warning: exported const PublicKeyTypePublicKeyTypeEd25519 should have comment (or a comment on this block) or be unexported (golint)
    • Line 10600: warning: receiver name s should be consistent with previous receiver name e for PublicKeyType (golint)
    • Line 10607: warning: receiver name s should be consistent with previous receiver name e for PublicKeyType (golint)
    • Line 10629: warning: exported const SignerKeyTypeSignerKeyTypeEd25519 should have comment (or a comment on this block) or be unexported (golint)
    • Line 10654: warning: receiver name s should be consistent with previous receiver name e for SignerKeyType (golint)
    • Line 10661: warning: receiver name s should be consistent with previous receiver name e for SignerKeyType (golint)
    • Line 10741: warning: receiver name s should be consistent with previous receiver name u for PublicKey (golint)
    • Line 10748: warning: receiver name s should be consistent with previous receiver name u for PublicKey (golint)
    • Line 10904: warning: receiver name s should be consistent with previous receiver name u for SignerKey (golint)
    • Line 10911: warning: receiver name s should be consistent with previous receiver name u for SignerKey (golint)
    • Line 10933: warning: receiver name s should be consistent with previous receiver name e for Signature (golint)
    • Line 10940: warning: receiver name s should be consistent with previous receiver name e for Signature (golint)
    • Line 10962: warning: receiver name s should be consistent with previous receiver name e for SignatureHint (golint)
    • Line 10969: warning: receiver name s should be consistent with previous receiver name e for SignatureHint (golint)
    • Line 11017: warning: receiver name s should be consistent with previous receiver name u for NodeId (golint)
    • Line 11024: warning: receiver name s should be consistent with previous receiver name u for NodeId (golint)
    • go/exp/crypto/derivation/main.go
    • Line 31: warning: exported var ErrInvalidPath should have comment or be unexported (golint)
    • Line 37: warning: exported type Key should have comment or be unexported (golint)
    • Line 87: warning: exported method Key.Derive should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/history_archive_state.go
    • Line 7: warning: exported const NumLevels should have comment or be unexported (golint)
    • Line 9: warning: exported type HistoryArchiveState should have comment or be unexported (golint)
    • Line 23: warning: exported method HistoryArchiveState.LevelSummary should have comment or be unexported (golint)
    • Line 44: warning: exported method HistoryArchiveState.Buckets should have comment or be unexported (golint)
    • Line 59: warning: exported method HistoryArchiveState.Range should have comment or be unexported (golint)
    • go/services/horizon/internal/txsub/errors.go
    • Line 12: warning: exported var ErrNoResults should have comment or be unexported (golint)
    • Line 35: warning: exported method FailedTransactionError.Result should have comment or be unexported (golint)
    • Line 35: warning: receiver name fte should be consistent with previous receiver name err for FailedTransactionError (golint)
    • Line 40: warning: exported method FailedTransactionError.TransactionResultCode should have comment or be unexported (golint)
    • Line 40: warning: receiver name fte should be consistent with previous receiver name err for FailedTransactionError (golint)
    • Line 50: warning: exported method FailedTransactionError.OperationResultCodes should have comment or be unexported (golint)
    • Line 50: warning: receiver name fte should be consistent with previous receiver name err for FailedTransactionError (golint)
    • go/tools/stellar-archivist/internal/scan.go
    • Line 26: warning: exported method Archive.ScanCheckpoints should have comment or be unexported (golint)
    • Line 37: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 42: warning: exported method Archive.ScanCheckpointsSlow should have comment or be unexported (golint)
    • Line 98: warning: exported method Archive.ScanCheckpointsFast should have comment or be unexported (golint)
    • Line 156: warning: exported method Archive.Scan should have comment or be unexported (golint)
    • Line 168: warning: exported method Archive.ScanAllBuckets should have comment or be unexported (golint)
    • Line 189: warning: exported method Archive.ScanBuckets should have comment or be unexported (golint)
    • Line 283: warning: exported method Archive.ClearCachedInfo should have comment or be unexported (golint)
    • Line 293: warning: exported method Archive.ReportCheckpointStats should have comment or be unexported (golint)
    • Line 304: warning: exported method Archive.ReportBucketStats should have comment or be unexported (golint)
    • Line 311: warning: exported method Archive.NoteCheckpointFile should have comment or be unexported (golint)
    • Line 317: warning: exported method Archive.NoteExistingBucket should have comment or be unexported (golint)
    • Line 323: warning: exported method Archive.NoteReferencedBucket should have comment or be unexported (golint)
    • Line 334: warning: exported method Archive.CheckCheckpointFilesMissing should have comment or be unexported (golint)
    • Line 350: warning: exported method Archive.CheckBucketsMissing should have comment or be unexported (golint)
    • Line 354: warning: should omit 2nd value from range; this loop is equivalent to `for k := range ...` (golint)
    • Line 363: warning: exported method Archive.ReportMissing should have comment or be unexported (golint)
    • Line 386: warning: should omit 2nd value from range; this loop is equivalent to `for bucket := range ...` (golint)
    • go/support/log/main.go
    • Line 16: warning: exported const PanicLevel should have comment (or a comment on this block) or be unexported (golint)
    • Line 79: warning: exported function SetLevel should have comment or be unexported (golint)
    • Line 83: warning: exported function WithField should have comment or be unexported (golint)
    • Line 88: warning: exported function WithFields should have comment or be unexported (golint)
    • Line 92: warning: exported function WithStack should have comment or be unexported (golint)
    • go/keypair/full.go
    • Line 11: warning: exported type Full should have comment or be unexported (golint)
    • Line 15: warning: exported method Full.Address should have comment or be unexported (golint)
    • Line 19: warning: exported method Full.Hint should have comment or be unexported (golint)
    • Line 24: warning: exported method Full.Seed should have comment or be unexported (golint)
    • Line 28: warning: exported method Full.Verify should have comment or be unexported (golint)
    • Line 42: warning: exported method Full.Sign should have comment or be unexported (golint)
    • Line 47: warning: exported method Full.SignDecorated should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/hash.go
    • Line 14: warning: exported type Hash should have comment or be unexported (golint)
    • Line 16: warning: exported function DecodeHash should have comment or be unexported (golint)
    • Line 23: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 27: warning: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...) (golint)
    • Line 36: warning: exported function MustDecodeHash should have comment or be unexported (golint)
    • Line 44: warning: exported function EmptyXdrArrayHash should have comment or be unexported (golint)
    • Line 51: warning: exported method Hash.IsZero should have comment or be unexported (golint)
    • go/services/horizon/internal/render/sse/stream.go
    • Line 20: warning: exported var ErrRateLimited should have comment or be unexported (golint)
    • Line 28: warning: exported type Stream should have comment or be unexported (golint)
    • Line 58: warning: exported method Stream.Send should have comment or be unexported (golint)
    • Line 66: warning: exported method Stream.SentCount should have comment or be unexported (golint)
    • Line 72: warning: exported method Stream.SetLimit should have comment or be unexported (golint)
    • Line 78: warning: exported method Stream.Done should have comment or be unexported (golint)
    • Line 103: warning: exported method Stream.Err should have comment or be unexported (golint)
    • go/services/horizon/internal/main_generated.go
    • Line 7: warning: exported method AggregateBalanceAction.Handle should have comment or be unexported (golint)
    • Line 13: warning: exported method ControlledAccountsAction.Handle should have comment or be unexported (golint)
    • Line 19: warning: exported method AccountShowAction.Handle should have comment or be unexported (golint)
    • Line 25: warning: exported method AssetsAction.Handle should have comment or be unexported (golint)
    • Line 31: warning: exported method DataShowAction.Handle should have comment or be unexported (golint)
    • Line 37: warning: exported method EffectIndexAction.Handle should have comment or be unexported (golint)
    • Line 43: warning: exported method LedgerIndexAction.Handle should have comment or be unexported (golint)
    • Line 49: warning: exported method LedgerShowAction.Handle should have comment or be unexported (golint)
    • Line 55: warning: exported method MetricsAction.Handle should have comment or be unexported (golint)
    • Line 61: warning: exported method NotFoundAction.Handle should have comment or be unexported (golint)
    • Line 67: warning: exported method NotImplementedAction.Handle should have comment or be unexported (golint)
    • Line 73: warning: exported method OffersByAccountAction.Handle should have comment or be unexported (golint)
    • Line 79: warning: exported method OperationFeeStatsAction.Handle should have comment or be unexported (golint)
    • Line 85: warning: exported method OperationIndexAction.Handle should have comment or be unexported (golint)
    • Line 91: warning: exported method OperationShowAction.Handle should have comment or be unexported (golint)
    • Line 97: warning: exported method OrderBookShowAction.Handle should have comment or be unexported (golint)
    • Line 103: warning: exported method PathIndexAction.Handle should have comment or be unexported (golint)
    • Line 109: warning: exported method PaymentsIndexAction.Handle should have comment or be unexported (golint)
    • Line 115: warning: exported method RateLimitExceededAction.Handle should have comment or be unexported (golint)
    • Line 121: warning: exported method RootAction.Handle should have comment or be unexported (golint)
    • Line 127: warning: exported method TradeAggregateIndexAction.Handle should have comment or be unexported (golint)
    • Line 133: warning: exported method TradeIndexAction.Handle should have comment or be unexported (golint)
    • Line 139: warning: exported method TransactionCreateAction.Handle should have comment or be unexported (golint)
    • Line 145: warning: exported method TransactionIndexAction.Handle should have comment or be unexported (golint)
    • Line 151: warning: exported method TransactionShowAction.Handle should have comment or be unexported (golint)
    • go/services/horizon/internal/db2/history/trade.go
    • Line 69: warning: comment on exported method TradesQ.ForAccount should be of the form "ForAccount ..." (golint)
    • Line 137: warning: exported method TradesQ.JoinAccounts should have comment or be unexported (golint)
    • Line 144: warning: exported method TradesQ.JoinAssets should have comment or be unexported (golint)
    • Line 213: warning: comment on exported method Q.InsertTrade should be of the form "InsertTrade ..." (golint)
    • Line 306: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • go/tools/stellar-archivist/internal/xdrstream.go
    • Line 20: warning: exported type XdrStream should have comment or be unexported (golint)
    • Line 26: warning: exported function NewXdrStream should have comment or be unexported (golint)
    • Line 30: warning: exported function NewXdrGzStream should have comment or be unexported (golint)
    • Line 39: warning: exported method Archive.GetXdrStream should have comment or be unexported (golint)
    • Line 50: warning: exported function HashXdr should have comment or be unexported (golint)
    • Line 60: warning: exported method XdrStream.Close should have comment or be unexported (golint)
    • Line 69: warning: exported method XdrStream.ReadOne should have comment or be unexported (golint)
    • Line 76: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 105: warning: exported function WriteFramedXdr should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/mock_archive.go
    • Line 16: warning: exported type MockArchiveBackend should have comment or be unexported (golint)
    • Line 21: warning: exported method MockArchiveBackend.Exists should have comment or be unexported (golint)
    • Line 28: warning: exported method MockArchiveBackend.GetFile should have comment or be unexported (golint)
    • Line 39: warning: exported method MockArchiveBackend.PutFile should have comment or be unexported (golint)
    • Line 50: warning: exported method MockArchiveBackend.ListFiles should have comment or be unexported (golint)
    • Line 56: warning: should omit 2nd value from range; this loop is equivalent to `for k := range ...` (golint)
    • Line 71: warning: exported method MockArchiveBackend.CanListFiles should have comment or be unexported (golint)
    • Line 75: warning: exported function MakeMockBackend should have comment or be unexported (golint)
    • go/protocols/horizon/operations/main.go
    • Line 11: warning: comment on exported var TypeNames should be of the form "TypeNames ..." (golint)
    • Line 48: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • go/keypair/from_address.go
    • Line 19: warning: exported method FromAddress.Address should have comment or be unexported (golint)
    • Line 23: warning: exported method FromAddress.Hint should have comment or be unexported (golint)
    • Line 28: warning: exported method FromAddress.Verify should have comment or be unexported (golint)
    • Line 42: warning: exported method FromAddress.Sign should have comment or be unexported (golint)
    • Line 46: warning: exported method FromAddress.SignDecorated should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/dirprefix.go
    • Line 12: warning: exported type DirPrefix should have comment or be unexported (golint)
    • Line 14: warning: exported method DirPrefix.Path should have comment or be unexported (golint)
    • Line 18: warning: exported method DirPrefix.PathPrefix should have comment or be unexported (golint)
    • Line 29: warning: exported function CheckpointPrefix should have comment or be unexported (golint)
    • Line 37: warning: exported function HashPrefix should have comment or be unexported (golint)
    • Line 41: warning: comment on exported function RangePaths should be of the form "RangePaths ..." (golint)
    • go/protocols/horizon/effects/main.go
    • Line 27: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 31: warning: exported type AccountCreated should have comment or be unexported (golint)
    • Line 36: warning: exported type AccountCredited should have comment or be unexported (golint)
    • Line 42: warning: exported type AccountDebited should have comment or be unexported (golint)
    • Line 48: warning: exported type AccountThresholdsUpdated should have comment or be unexported (golint)
    • Line 55: warning: exported type AccountHomeDomainUpdated should have comment or be unexported (golint)
    • Line 60: warning: exported type AccountFlagsUpdated should have comment or be unexported (golint)
    • Line 66: warning: exported type SequenceBumped should have comment or be unexported (golint)
    • Line 71: warning: exported type SignerCreated should have comment or be unexported (golint)
    • Line 78: warning: exported type SignerRemoved should have comment or be unexported (golint)
    • Line 85: warning: exported type SignerUpdated should have comment or be unexported (golint)
    • Line 92: warning: exported type TrustlineCreated should have comment or be unexported (golint)
    • Line 98: warning: exported type TrustlineRemoved should have comment or be unexported (golint)
    • Line 104: warning: exported type TrustlineUpdated should have comment or be unexported (golint)
    • Line 110: warning: exported type TrustlineAuthorized should have comment or be unexported (golint)
    • Line 117: warning: exported type TrustlineDeauthorized should have comment or be unexported (golint)
    • Line 124: warning: exported type Trade should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/range.go
    • Line 13: warning: exported const CheckpointFreq should have comment or be unexported (golint)
    • Line 15: warning: exported type Range should have comment or be unexported (golint)
    • Line 20: warning: exported function PrevCheckpoint should have comment or be unexported (golint)
    • Line 28: warning: exported function NextCheckpoint should have comment or be unexported (golint)
    • Line 41: warning: exported function MakeRange should have comment or be unexported (golint)
    • Line 51: warning: exported method Range.Clamp should have comment or be unexported (golint)
    • Line 67: warning: exported method Range.Checkpoints should have comment or be unexported (golint)
    • Line 78: warning: exported method Range.Size should have comment or be unexported (golint)
    • Line 82: warning: exported method Range.CollapsedString should have comment or be unexported (golint)
    • Line 85: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 90: warning: exported type ByUint32 should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/verify.go
    • Line 33: warning: exported type ByHash should have comment or be unexported (golint)
    • Line 47: warning: exported function SortTxsForHash should have comment or be unexported (golint)
    • Line 63: warning: exported function HashTxSet should have comment or be unexported (golint)
    • Line 83: warning: exported function HashEmptyTxSet should have comment or be unexported (golint)
    • Line 87: warning: exported method Archive.VerifyLedgerHeaderHistoryEntry should have comment or be unexported (golint)
    • Line 107: warning: exported method Archive.VerifyTransactionHistoryEntry should have comment or be unexported (golint)
    • Line 118: warning: exported method Archive.VerifyTransactionHistoryResultEntry should have comment or be unexported (golint)
    • Line 129: warning: exported method Archive.VerifyCategoryCheckpoint should have comment or be unexported (golint)
    • Line 205: warning: exported method Archive.VerifyBucketHash should have comment or be unexported (golint)
    • Line 220: warning: exported method Archive.VerifyBucketEntries should have comment or be unexported (golint)
    • Line 271: warning: exported method Archive.ReportInvalid should have comment or be unexported (golint)
    • Line 280: warning: should omit 2nd value from range; this loop is equivalent to `for i := range ...` (golint)
    • go/protocols/horizon/main.go
    • Line 1: warning: package comment should be of the form "Package horizon ..." (golint)
    • Line 77: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 87: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 173: warning: exported method Ledger.PagingToken should have comment or be unexported (golint)
    • Line 173: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 196: warning: exported method Offer.PagingToken should have comment or be unexported (golint)
    • Line 196: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 221: warning: comment on exported method Path.PagingToken should be of the form "PagingToken ..." (golint)
    • Line 222: warning: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (golint)
    • Line 404: warning: receiver name res should be consistent with previous receiver name t for Transaction (golint)
    • Line 455: warning: exported type AggregateBalanceRecord should have comment or be unexported (golint)
    • Line 460: warning: exported type AggregatedBalances should have comment or be unexported (golint)
    • Line 475: warning: exported type ControlledAccountRecord should have comment or be unexported (golint)
    • Line 480: warning: exported type ControlledAccounts should have comment or be unexported (golint)
    • go/xdr/asset.go
    • Line 13: warning: exported function MustNewNativeAsset should have comment or be unexported (golint)
    • Line 22: warning: exported function MustNewCreditAsset should have comment or be unexported (golint)
    • go/support/log/loggly_hook.go
    • Line 26: warning: exported method LogglyHook.Fire should have comment or be unexported (golint)
    • Line 46: warning: exported method LogglyHook.Flush should have comment or be unexported (golint)
    • Line 50: warning: exported method LogglyHook.Levels should have comment or be unexported (golint)
    • go/tools/stellar-archivist/main.go
    • Line 35: warning: exported type Options should have comment or be unexported (golint)
    • Line 44: warning: exported method Options.SetRange should have comment or be unexported (golint)
    • Line 60: warning: exported method Options.MaybeProfile should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/s3_archive.go
    • Line 17: warning: exported type S3ArchiveBackend should have comment or be unexported (golint)
    • Line 23: warning: exported method S3ArchiveBackend.GetFile should have comment or be unexported (golint)
    • Line 35: warning: exported method S3ArchiveBackend.Exists should have comment or be unexported (golint)
    • Line 44: warning: exported method S3ArchiveBackend.PutFile should have comment or be unexported (golint)
    • Line 62: warning: exported method S3ArchiveBackend.ListFiles should have comment or be unexported (golint)
    • Line 100: warning: exported method S3ArchiveBackend.CanListFiles should have comment or be unexported (golint)
    • Line 104: warning: exported function MakeS3Backend should have comment or be unexported (golint)
    • go/xdr/main.go
    • Line 60: warning: exported function MarshalBase64 should have comment or be unexported (golint)
    • go/services/horizon/internal/ingest/batch_insert_builder.go
    • Line 19: warning: exported method BatchInsertBuilder.GetAddresses should have comment or be unexported (golint)
    • Line 30: warning: exported method BatchInsertBuilder.ReplaceAddressesWithIDs should have comment or be unexported (golint)
    • Line 40: warning: exported method BatchInsertBuilder.Values should have comment or be unexported (golint)
    • Line 45: warning: exported method BatchInsertBuilder.Exec should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/archive.go
    • Line 24: warning: exported type CommandOptions should have comment or be unexported (golint)
    • Line 33: warning: exported type ConnectOptions should have comment or be unexported (golint)
    • Line 38: warning: exported type ArchiveBackend should have comment or be unexported (golint)
    • Line 46: warning: exported type Archive should have comment or be unexported (golint)
    • Line 69: warning: exported method Archive.GetPathHAS should have comment or be unexported (golint)
    • Line 81: warning: exported method Archive.PutPathHAS should have comment or be unexported (golint)
    • Line 94: warning: exported method Archive.BucketExists should have comment or be unexported (golint)
    • Line 98: warning: exported method Archive.CategoryCheckpointExists should have comment or be unexported (golint)
    • Line 102: warning: exported method Archive.GetRootHAS should have comment or be unexported (golint)
    • Line 106: warning: exported method Archive.GetCheckpointHAS should have comment or be unexported (golint)
    • Line 110: warning: exported method Archive.PutCheckpointHAS should have comment or be unexported (golint)
    • Line 114: warning: exported method Archive.PutRootHAS should have comment or be unexported (golint)
    • Line 122: warning: exported method Archive.ListBucket should have comment or be unexported (golint)
    • Line 126: warning: exported method Archive.ListAllBuckets should have comment or be unexported (golint)
    • Line 130: warning: exported method Archive.ListAllBucketHashes should have comment or be unexported (golint)
    • Line 147: warning: exported method Archive.ListCategoryCheckpoints should have comment or be unexported (golint)
    • Line 172: warning: exported function Connect should have comment or be unexported (golint)
    • Line 211: warning: exported function MustConnect should have comment or be unexported (golint)
    • go/tools/stellar-archivist/internal/http_archive.go
    • Line 16: warning: exported type HttpArchiveBackend should have comment or be unexported (golint)
    • Line 24: warning: if block ends with a return statement, so drop this else and outdent its block (golint)
    • Line 30: warning: exported method HttpArchiveBackend.GetFile should have comment or be unexported (golint)
    • Line 50: warning: exported method HttpArchiveBackend.Exists should have comment or be unexported (golint)
    • Line 60: warning: exported method HttpArchiveBackend.PutFile should have comment or be unexported (golint)
    • Line 65: warning: exported method HttpArchiveBackend.ListFiles should have comment or be unexported (golint)
    • Line 74: warning: exported method HttpArchiveBackend.CanListFiles should have comment or be unexported (golint)
    • Line 78: warning: exported function MakeHttpBackend should have comment or be unexported (golint)
    • go/services/horizon/internal/db2/core/main.go
    • Line 12: warning: exported type AggregateBalanceString should have comment or be unexported (golint)
    • Line 17: warning: exported method AggregateBalanceString.PagingToken should have comment or be unexported (golint)
    • Line 21: warning: exported type ControlledAccountString should have comment or be unexported (golint)
    • Line 26: warning: exported method ControlledAccountString.PagingToken should have comment or be unexported (golint)

gocyclo98%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

    • go/xdr/xdr_generated.go
    • Line 5221: warning: cyclomatic complexity 26 of function NewStellarMessage() is high (> 15) (gocyclo)
    • Line 9542: warning: cyclomatic complexity 25 of function NewOperationResultTr() is high (> 15) (gocyclo)
    • Line 6452: warning: cyclomatic complexity 24 of function NewOperationBody() is high (> 15) (gocyclo)
    • go/xdr/asset.go
    • Line 113: warning: cyclomatic complexity 17 of function (Asset).Extract() is high (> 15) (gocyclo)

ineffassign97%

IneffAssign detects ineffectual assignments in Go code.


license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


misspell95%

Misspell Finds commonly misspelled English words

    • go/xdr/xdr_generated.go
    • Line 7633: warning: "successfuly" is a misspelling of "successfully" (misspell)
    • Line 8897: warning: "transfered" is a misspelling of "transferred" (misspell)
    • Line 10088: warning: "occured" is a misspelling of "occurred" (misspell)