Skip to content

ci: automated update to rustc 1.89.0 #298

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

create-pr-actions[bot]
Copy link

Automated update to Github CI workflow cont_integration.yml by create-pull-request GitHub action

@ValuedMammal
Copy link
Collaborator

Some lifetimes were hidden. Using a named (or anonymous) lifetime seems to fix it.

diff --git a/wallet/src/descriptor/policy.rs b/wallet/src/descriptor/policy.rs
index eb9014cb..100348f6 100644
--- a/wallet/src/descriptor/policy.rs
+++ b/wallet/src/descriptor/policy.rs
@@ -1053,9 +1053,9 @@ impl<Ctx: ScriptContext + 'static> ExtractPolicy for Miniscript<DescriptorPublic
         })
     }
 }
 
-fn psbt_inputs_sat(psbt: &Psbt) -> impl Iterator<Item = PsbtInputSatisfier> {
+fn psbt_inputs_sat(psbt: &Psbt) -> impl Iterator<Item = PsbtInputSatisfier<'_>> {
     (0..psbt.inputs.len()).map(move |i| PsbtInputSatisfier::new(psbt, i))
 }
 
 /// Options to build the satisfaction field in the policy
diff --git a/wallet/src/wallet/mod.rs b/wallet/src/wallet/mod.rs
index c1aab082..e1f957cd 100644
--- a/wallet/src/wallet/mod.rs
+++ b/wallet/src/wallet/mod.rs
@@ -1164,9 +1164,9 @@ impl Wallet {
     /// }
     /// ```
     ///
     /// [`Anchor`]: bdk_chain::Anchor
-    pub fn get_tx(&self, txid: Txid) -> Option<WalletTx> {
+    pub fn get_tx(&self, txid: Txid) -> Option<WalletTx<'_>> {
         let graph = self.indexed_graph.graph();
         graph
             .list_canonical_txs(
                 &self.chain,
@@ -1186,9 +1186,9 @@ impl Wallet {
     /// [`TxGraph::full_txs`].
     ///
     /// To iterate over all canonical transactions, including those that are irrelevant, use
     /// [`TxGraph::list_canonical_txs`].
-    pub fn transactions(&self) -> impl Iterator<Item = WalletTx> + '_ {
+    pub fn transactions<'a>(&'a self) -> impl Iterator<Item = WalletTx<'a>> + 'a {
         let tx_graph = self.indexed_graph.graph();
         let tx_index = &self.indexed_graph.index;
         tx_graph
             .list_canonical_txs(
@@ -1214,9 +1214,9 @@ impl Wallet {
     /// let sorted_txs: Vec<WalletTx> =
     ///     wallet.transactions_sort_by(|tx1, tx2| tx2.chain_position.cmp(&tx1.chain_position));
     /// # Ok::<(), anyhow::Error>(())
     /// ```
-    pub fn transactions_sort_by<F>(&self, compare: F) -> Vec<WalletTx>
+    pub fn transactions_sort_by<F>(&self, compare: F) -> Vec<WalletTx<'_>>
     where
         F: FnMut(&WalletTx, &WalletTx) -> Ordering,
     {

@bitcoindevkit bitcoindevkit deleted a comment from coveralls Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants