NPO-Accounting/conservancy_beancount Changeset - 52fa66bba1e8

Changeset - 52fa66bba1e8

Parent rev.

Child rev.

[Not reviewed]

0 1 0

Brett Smith - 4 years ago 2020-05-28 13:01:00
brettcsmith@brettcsmith.org

data: Specify the type of Posting.cost.

The rationale is the same as it was for Posting.amount.

1 file changed with 2 insertions and 0 deletions:

conservancy_beancount/data.py

0 comments (0 inline, 0 general)

conservancy_beancount/data.py

➞

Show inline comments

@@ ... / @@ -24,12 +24,13 @@ import datetime @@
 import decimal
 import functools
 from beancount.core import account as bc_account
 from beancount.core import amount as bc_amount
 from beancount.core import convert as bc_convert
 from beancount.core import position as bc_position
 from typing import (
     cast,
     Callable,
     Hashable,
     Iterable,
@@ ... / @@ -266,12 +267,13 @@ class Posting(BasePosting): @@
     * The `meta` field is a PostingMeta object
     """
     __slots__ = ()
     account: Account
     units: Amount
     cost: Optional[bc_position.Cost]
     # mypy correctly complains that our MutableMapping is not compatible
     # with Beancount's meta type declaration of Optional[Dict]. IMO
     # Beancount's type declaration is a smidge too specific: I think its type
     # declaration should also use MutableMapping, because it would be very
     # unusual for code to specifically require a Dict over that.
     # If it did, this declaration would pass without issue.

0 comments (0 inline, 0 general)