Skip to content

fix: don't check Synchrony params when PBTS is not enabled#2686

Merged
cason merged 6 commits intomainfrom
cason/2580-synch-params
Mar 28, 2024
Merged

fix: don't check Synchrony params when PBTS is not enabled#2686
cason merged 6 commits intomainfrom
cason/2580-synch-params

Conversation

@cason
Copy link

@cason cason commented Mar 28, 2024

Closes #2580.


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments
  • Title follows the Conventional Commits spec

@cason cason added this to the 2024-Q1 milestone Mar 28, 2024
@cason cason self-assigned this Mar 28, 2024
@cason cason requested a review from a team as a code owner March 28, 2024 08:33
@cason cason requested a review from a team March 28, 2024 08:33
@cason cason changed the title feat: don't check Synchrony params when PBTS is not enabled fix: don't check Synchrony params when PBTS is not enabled Mar 28, 2024
@cason cason added the pbts label Mar 28, 2024
Daniel and others added 2 commits March 28, 2024 12:04
Copy link
Collaborator

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

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

🙏

@cason cason added this pull request to the merge queue Mar 28, 2024
Merged via the queue into main with commit b9e3194 Mar 28, 2024
@cason cason deleted the cason/2580-synch-params branch March 28, 2024 16:48
mergify bot pushed a commit that referenced this pull request Mar 28, 2024
Closes  #2580.

---

#### PR checklist

- [x] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments
- [x] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec

---------

Co-authored-by: lasaro <lasaro@informal.systems>
(cherry picked from commit b9e3194)
cason pushed a commit that referenced this pull request Mar 28, 2024
…2686) (#2690)

Closes  #2580.

---

#### PR checklist

- [x] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments
- [x] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
<hr>This is an automatic backport of pull request #2686 done by
[Mergify](https://mergify.com).

Co-authored-by: Daniel <daniel.cason@informal.systems>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

PBTS: Don't check synchrony parameters when PBTS is not enabled

3 participants