The labeler wasn't applying `C-Protocol-Critical` labels correctly. I misinterpreted how the `any` rule was supposed to work.

First commit in a series of commits to clean up the contract tests. Made
a deploy helper function to remove some ugly nested awaits. Also
generally starting to remove unnecessary Factory objects.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Version Packages

Develop -> Master PR

changesets: delete old batch submitter

Develop -> Master PR

* Revert "l2geth: Publish transactions to external msg queue"

This reverts commits:
* 9f05c877^..f8348862
* 3e016ae8^..36991e68

* add changeet
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* go-builder: bump versions

Use `golang:1.18.0-alpine3.15`
and `golangci-lint@1.45.2`. This
will allow linting of code that
uses generics

* ci: add go-builder + js-builder to ci publish
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

This should speed up builds as more common
base images are built and also opens
up the possibility of using generics in the code.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

The tests include:
- `TestBoundedForkedSync62`
- `TestBoundedForkedSync63Full`
- `TestBoundedForkedSync63Fast`
- `TestBoundedForkedSync64Full`
- `TestBoundedForkedSync64Fast`
- `TestBoundedForkedSync64Light`

None of these tests cover code that we use in production.
These tests were flaky upstream and eventually fixed.
No major changes to this codebase are planned before
deprecating it.
Co-authored-by: Matthew Slipper <me@matthewslipper.com>

ci: Fix bad condition

Not sure why Mergify didn't pick this up in CI.

- Requires all threads to be resolved
- Adds/removes a `conflict` label when there are merge conflicts/when they are cleared.
- Adds a comment and removes the `on-merge-train` label when a queued merge fails.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Teleportr was getting released on GitHub, but not pushed to Docker Hub. The indexer's canary builds were pushing `batch-submitter-service` images, which was incorrect.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

* deps: bump to typescript@4.6.2

This bumps the typescript dep so that only one needs
to be installed and updates to the latest version
of typescript

* Update tsconfigs
Co-authored-by: Matthew Slipper <me@matthewslipper.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Co-authored-by: Maurelian <maurelian@protonmail.ch>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

l2geth: skip flaky accountcmd tests

Develop -> Master Release PR

Version Packages

Develop -> Master

* fix(dtl): kovan DTL halting issue

* fix(dtl): add consistency check for L2 sync

* dtl: add retry logic on startup

* dtl: fix startup
Co-authored-by: Mark Tyneway <mark.tyneway@gmail.com>

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Skip some tests that flake in CI.
It is safe to skip these because
this code will not be changed and we
will be deprecating this codebase soon.

* feat(ci): check deploy markdown correctness

Adds a step to the contract tests to confirm that the contract
deployment docs are up to date. Will only fail if a deployment has been
modified but yarn autogen:markdown was not run properly.

* maint(ct): update deployments markdown

Updates the deployments markdown files. Should've been updated after we
deployed the Teleportr contracts to Kovan but we didn't have the CI
workflow in place to detect that this didn't happen.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Previously the chainid was too large for the ledger package to
work. This is due to ethers using an old version of the ledger
package. There is an unresolved open issue here:
https://github.com/ethers-io/ethers.js/issues/1365

This updates the chainid to 17 to get around this issue.
This chain id is used by ThaiChain 2.0 ThaiFi which
has very little activity.
https://exp.thaifi.com/

It also lints the docker compose file to use `'`
consistently
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Adds a new feature to BaseServiceV2 to have a service exit immediately
when 3 exit signals are received. Useful when the main loop would take a
long time to exit but you want the service to exit immediately.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Adds a new JsonRpcProvider validator which specifically parses the input
type into a JsonRpcProvider instead of a generic provider.
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

ci: Merge using merge queue

l2geth: Skip another flaky test

This code is not going to change before deprecation
of `l2geth` in favor of bedrock. Skip a couple of tests
that fail in CI that are not involved at all in consensus.

ci: Add Mergify