Skip to content

feature/accounting - #60

Draft
decanus wants to merge 3 commits into
masterfrom
feature/accounting
Draft

feature/accounting#60
decanus wants to merge 3 commits into
masterfrom
feature/accounting

Conversation

@decanus

@decanus decanus commented Sep 17, 2019

Copy link
Copy Markdown
Contributor

Experimental implementation of specs/#1

Comment thread Sources/UB/Accounting/Accounting.swift Outdated
///
/// - Parameters:
/// - message: The message to account for.
func account(sentMessage message: Message);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines should not have trailing semicolons.

Comment thread Sources/UB/Accounting/Accounting.swift Outdated
///
/// - Parameters:
/// - message: The message to account for.
func account(receivedMessage message: Message);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines should not have trailing semicolons.

Comment thread Sources/UB/Accounting/Accounting.swift Outdated
/// - sentMessagesForPeer: The peer id to check.
///
/// - Returns: The amount of messages accounted for.
func accounted(sentMessagesForPeer peer: Peer) -> UInt;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines should not have trailing semicolons.

Comment thread Sources/UB/Accounting/Accounting.swift Outdated
/// - receivedMessagesForPeer: The peer id to check.
///
/// - Returns: The amount of messages accounted for.
func accounted(receivedMessagesForPeer peer: Peer) -> UInt;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lines should not have trailing semicolons.

@qlty-cloud-legacy

Copy link
Copy Markdown

Code Climate has analyzed commit e8714a0 and detected 0 issues on this pull request.

View more on Code Climate.

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.

1 participant