fix: remove # type: ignore after each REQUIRED

tools/generate_tx_models.py

@@ -117,7 +117,7 @@ def _generate_param_line(param: str, is_required: bool) -> str:
             param_name = param[2:]
             param_type = sfields[param_name][0]
             if is_required:
-                param_type_output = f"{TYPE_MAP[param_type]} = REQUIRED  # type: ignore"
+                param_type_output = f"{TYPE_MAP[param_type]} = REQUIRED"
             else:
                 param_type_output = f"Optional[{TYPE_MAP[param_type]}] = None"
             return f"    {_key_to_json(param_name)}: {param_type_output}"

xrpl/models/amounts/issued_currency_amount.py

@@ -25,7 +25,7 @@ class IssuedCurrencyAmount(IssuedCurrency):
     See https://xrpl.org/currency-formats.html#issued-currency-amounts.
     """
 
-    value: Union[str, int, float] = REQUIRED  # type: ignore
+    value: Union[str, int, float] = REQUIRED
     """
     This field is required.
 

xrpl/models/auth_account.py

@@ -14,7 +14,7 @@
 class AuthAccount(NestedModel):
     """Represents one entry in a list of AuthAccounts used in AMMBid transaction."""
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/currencies/issued_currency.py

@@ -36,14 +36,14 @@ class IssuedCurrency(BaseModel):
     See https://xrpl.org/currency-formats.html#specifying-currency-amounts
     """
 
-    currency: str = REQUIRED  # type: ignore
+    currency: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    issuer: str = REQUIRED  # type: ignore
+    issuer: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/account_channels.py

@@ -30,7 +30,7 @@ class AccountChannels(Request, LookupByLedgerRequest):
     """
 
     method: RequestMethod = field(default=RequestMethod.ACCOUNT_CHANNELS, init=False)
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/account_currencies.py

@@ -28,7 +28,7 @@ class AccountCurrencies(Request, LookupByLedgerRequest):
     `See account_currencies <https://xrpl.org/account_currencies.html>`_
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/account_info.py

@@ -26,7 +26,7 @@ class AccountInfo(Request, LookupByLedgerRequest):
     `See account_info <https://xrpl.org/account_info.html>`_
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/account_lines.py

@@ -25,7 +25,7 @@ class AccountLines(Request, LookupByLedgerRequest):
     `See account_lines <https://xrpl.org/account_lines.html>`_
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/account_nfts.py

@@ -17,7 +17,7 @@ class AccountNFTs(Request, LookupByLedgerRequest):
     """
 
     method: RequestMethod = field(default=RequestMethod.ACCOUNT_NFTS, init=False)
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     The unique identifier of an account, typically the account's address. The
     request returns NFTs owned by this account. This value is required.

xrpl/models/requests/account_objects.py

@@ -48,7 +48,7 @@ class AccountObjects(Request, LookupByLedgerRequest):
     `See account_objects <https://xrpl.org/account_objects.html>`_
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/account_offers.py

@@ -23,7 +23,7 @@ class AccountOffers(Request, LookupByLedgerRequest):
     `See account_offers <https://xrpl.org/account_offers.html>`_
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/account_tx.py

@@ -23,7 +23,7 @@ class AccountTx(Request, LookupByLedgerRequest):
     `See account_tx <https://xrpl.org/account_tx.html>`_
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/book_offers.py

@@ -20,14 +20,14 @@ class BookOffers(Request, LookupByLedgerRequest):
     as the order book, between two currencies.
     """
 
-    taker_gets: Currency = REQUIRED  # type: ignore
+    taker_gets: Currency = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    taker_pays: Currency = REQUIRED  # type: ignore
+    taker_pays: Currency = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/channel_authorize.py

@@ -46,14 +46,14 @@ class ChannelAuthorize(Request):
     """
 
     method: RequestMethod = field(default=RequestMethod.CHANNEL_AUTHORIZE, init=False)
-    channel_id: str = REQUIRED  # type: ignore
+    channel_id: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    amount: str = REQUIRED  # type: ignore
+    amount: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/channel_verify.py

@@ -21,28 +21,28 @@ class ChannelVerify(Request):
     """
 
     method: RequestMethod = field(default=RequestMethod.CHANNEL_VERIFY, init=False)
-    channel_id: str = REQUIRED  # type: ignore
+    channel_id: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    amount: str = REQUIRED  # type: ignore
+    amount: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    public_key: str = REQUIRED  # type: ignore
+    public_key: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    signature: str = REQUIRED  # type: ignore
+    signature: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/deposit_authorized.py

@@ -22,14 +22,14 @@ class DepositAuthorized(Request, LookupByLedgerRequest):
     authorization to deliver money to your account.
     """
 
-    source_account: str = REQUIRED  # type: ignore
+    source_account: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    destination_account: str = REQUIRED  # type: ignore
+    destination_account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/gateway_balances.py

@@ -23,7 +23,7 @@ class GatewayBalances(Request, LookupByLedgerRequest):
     `See gateway_balances <https://xrpl.org/gateway_balances.html>`_
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/get_aggregate_price.py

@@ -26,13 +26,13 @@ class GetAggregatePrice(Request):
 
     method: RequestMethod = field(default=RequestMethod.GET_AGGREGATE_PRICE, init=False)
 
-    base_asset: str = REQUIRED  # type: ignore
+    base_asset: str = REQUIRED
     """The currency code of the asset to be priced"""
 
-    quote_asset: str = REQUIRED  # type: ignore
+    quote_asset: str = REQUIRED
     """The currency code of the asset to quote the price of the base asset"""
 
-    oracles: List[Oracle] = REQUIRED  # type: ignore
+    oracles: List[Oracle] = REQUIRED
     """The oracle identifier"""
 
     trim: Optional[int] = None

xrpl/models/requests/ledger_entry.py

@@ -51,14 +51,14 @@ class DepositPreauth(BaseModel):
     object ID.
     """
 
-    owner: str = REQUIRED  # type: ignore
+    owner: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    authorized: str = REQUIRED  # type: ignore
+    authorized: str = REQUIRED
     """
     This field is required.
 
@@ -74,14 +74,14 @@ class Directory(BaseModel):
     object ID.
     """
 
-    owner: str = REQUIRED  # type: ignore
+    owner: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    dir_root: str = REQUIRED  # type: ignore
+    dir_root: str = REQUIRED
     """
     This field is required.
 
@@ -98,14 +98,14 @@ class Escrow(BaseModel):
     object ID.
     """
 
-    owner: str = REQUIRED  # type: ignore
+    owner: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    seq: int = REQUIRED  # type: ignore
+    seq: int = REQUIRED
     """
     This field is required.
 
@@ -121,14 +121,14 @@ class Offer(BaseModel):
     object ID.
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    seq: int = REQUIRED  # type: ignore
+    seq: int = REQUIRED
     """
     This field is required.
 
@@ -144,14 +144,14 @@ class Oracle(BaseModel):
     object ID.
     """
 
-    account: str = REQUIRED  # type: ignore
+    account: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    oracle_document_id: Union[str, int] = REQUIRED  # type: ignore
+    oracle_document_id: Union[str, int] = REQUIRED
     """
     This field is required.
 
@@ -164,14 +164,14 @@ class Oracle(BaseModel):
 class RippleState(BaseModel):
     """Required fields for requesting a RippleState if not querying by object ID."""
 
-    accounts: List[str] = REQUIRED  # type: ignore
+    accounts: List[str] = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    currency: str = REQUIRED  # type: ignore
+    currency: str = REQUIRED
     """
     This field is required.
 
@@ -184,14 +184,14 @@ class RippleState(BaseModel):
 class Ticket(BaseModel):
     """Required fields for requesting a Ticket if not querying by object ID."""
 
-    owner: str = REQUIRED  # type: ignore
+    owner: str = REQUIRED
     """
     This field is required.
 
     :meta hide-value:
     """
 
-    ticket_sequence: int = REQUIRED  # type: ignore
+    ticket_sequence: int = REQUIRED
     """
     This field is required.
 
@@ -204,7 +204,7 @@ class Ticket(BaseModel):
 class XChainClaimID(XChainBridge):
     """Required fields for requesting an XChainClaimID if not querying by object ID."""
 
-    xchain_claim_id: Union[int, str] = REQUIRED  # type: ignore
+    xchain_claim_id: Union[int, str] = REQUIRED
     """
     The `XChainClaimID` associated with a cross-chain transfer, which was created in an
     `XChainCreateClaimID` transaction. This field is required.
@@ -221,7 +221,7 @@ class XChainCreateAccountClaimID(XChainBridge):
     object ID.
     """
 
-    xchain_create_account_claim_id: Union[int, str] = REQUIRED  # type: ignore
+    xchain_create_account_claim_id: Union[int, str] = REQUIRED
     """
     The `XChainCreateAccountClaimID` associated with a cross-chain account create. This
     field is required.

xrpl/models/requests/manifest.py

@@ -23,7 +23,7 @@ class Manifest(Request):
     """
 
     method: RequestMethod = field(default=RequestMethod.MANIFEST, init=False)
-    public_key: str = REQUIRED  # type: ignore
+    public_key: str = REQUIRED
     """
     This field is required.
 

xrpl/models/requests/nft_buy_offers.py

@@ -19,7 +19,7 @@ class NFTBuyOffers(Request, LookupByLedgerRequest):
     """
 
     method: RequestMethod = field(default=RequestMethod.NFT_BUY_OFFERS, init=False)
-    nft_id: str = REQUIRED  # type: ignore
+    nft_id: str = REQUIRED
     """
     The unique identifier of an NFToken.
     The request returns buy offers for this NFToken. This value is required.

xrpl/models/requests/nft_history.py

@@ -20,7 +20,7 @@ class NFTHistory(Request, LookupByLedgerRequest):
     """
 
     method: RequestMethod = field(default=RequestMethod.NFT_HISTORY, init=False)
-    nft_id: str = REQUIRED  # type: ignore
+    nft_id: str = REQUIRED
     """
     The unique identifier of an NFToken.
     The request returns past transactions of this NFToken. This value is required.

xrpl/models/requests/nft_info.py

@@ -19,7 +19,7 @@ class NFTInfo(Request, LookupByLedgerRequest):
     """
 
     method: RequestMethod = field(default=RequestMethod.NFT_INFO, init=False)
-    nft_id: str = REQUIRED  # type: ignore
+    nft_id: str = REQUIRED
     """
     The unique identifier of an NFToken.
     The request returns information of this NFToken. This value is required.