Skip to content

chore: rename invalid block num to invalid max block num#13894

Merged
nventuro merged 1 commit intomasterfrom
nv/error-bn-msg
May 21, 2025
Merged

chore: rename invalid block num to invalid max block num#13894
nventuro merged 1 commit intomasterfrom
nv/error-bn-msg

Conversation

@nventuro
Copy link
Contributor

@nventuro nventuro commented Apr 28, 2025

We currently error out with 'invalid block number' when we detect a tx's max block number as being invalid, I updated the error message to clarify this. I assume we don't consider this a breaking change.

Closes #13622

@nventuro nventuro requested a review from spalladino April 28, 2025 18:46
@nventuro nventuro enabled auto-merge April 28, 2025 19:39
@nventuro nventuro requested review from alexghr and removed request for spalladino May 21, 2025 10:13
@nventuro nventuro added this pull request to the merge queue May 21, 2025
Merged via the queue into master with commit 0db31ce May 21, 2025
5 checks passed
@nventuro nventuro deleted the nv/error-bn-msg branch May 21, 2025 11:05
charlielye pushed a commit that referenced this pull request May 21, 2025
🤖 I have created a new Aztec Packages release
---


##
[0.87.2](v0.87.1...v0.87.2)
(2025-05-21)


### Miscellaneous

* improved readability
AztecNodeService.getL2ToL1MessageMembershipWitness
([#14273](#14273))
([038a518](038a518))
* rename invalid block num to invalid max block num
([#13894](#13894))
([0db31ce](0db31ce))
* syncNotes --> syncPrivateState
([#14438](#14438))
([845830a](845830a)),
closes
[#13113](#13113)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Thunkar pushed a commit that referenced this pull request May 23, 2025
We currently error out with 'invalid block number' when we detect a tx's
_max_ block number as being invalid, I updated the error message to
clarify this. I assume we don't consider this a breaking change.

Closes #13622
Thunkar pushed a commit that referenced this pull request May 23, 2025
🤖 I have created a new Aztec Packages release
---


##
[0.87.2](v0.87.1...v0.87.2)
(2025-05-21)


### Miscellaneous

* improved readability
AztecNodeService.getL2ToL1MessageMembershipWitness
([#14273](#14273))
([038a518](038a518))
* rename invalid block num to invalid max block num
([#13894](#13894))
([0db31ce](0db31ce))
* syncNotes --> syncPrivateState
([#14438](#14438))
([845830a](845830a)),
closes
[#13113](#13113)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

shared mutable error change

2 participants