[Tooling] fix PR template testing checklist item formatting #268
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bryanchriswhite
changed the title
bryanchriswhite
added
the
tooling
h5law
added a commit
that referenced
this pull request
Dec 14, 2023
commit d621631 Author: Redouane Lakrache <[email protected]> Date: Wed Dec 13 16:24:34 2023 +0100 [SDK] feat: Have distinct JSON-RPC and gRPC urls (#261) * feat: Have distinct JSON-RPC and gRPC urls * chore: Trigger e2e tests * chore: Fix import groups commit 4e30e27 Author: Bryan White <[email protected]> Date: Wed Dec 13 14:28:36 2023 +0100 fix PR template testing checklist item (#268)
h5law
added a commit
that referenced
this pull request
Dec 14, 2023
commit df73cfa Author: Bryan White <[email protected]> Date: Thu Dec 14 01:06:03 2023 +0100 refactor: `NewMinedRelay` to shared testutil (#262) commit 410965a Author: Bryan White <[email protected]> Date: Thu Dec 14 01:05:52 2023 +0100 fix: PR template typo 2 (#269) commit dd7df68 Author: Daniel Olshansky <[email protected]> Date: Wed Dec 13 13:45:26 2023 -0800 [Testing] Unit tests, E2E tests, logging and other Fixes / "Touchups" (#253) This is aiming to fix multiple issues in E2E tests and just general QoL improvements: 1. Adding a comment in `relay.feature` about the need to run `make supplier1_stake && make app1_stake` until we fix #180 (known issue) 2. Replacing the reference to host `sequencer-poktroll-sequencer` with `localhost` (see first screenshot below) 3. Fixed a bug leading us to assert against a `MsgSubmitProof` event instead of a `MsgCreateClaim` event (see second screenshot below) 4. Fixed on-chain logging under `x/*` by replacing `logger\.(Info|Error)\("([^"]+)"(?:, (.*))?\)` with `logger.$1(fmt.Sprintf("$2", $3))` so it works as expected 5. Some quality-of-life comments & TODOs 6. Minor additions to unit tests from #220 commit d621631 Author: Redouane Lakrache <[email protected]> Date: Wed Dec 13 16:24:34 2023 +0100 [SDK] feat: Have distinct JSON-RPC and gRPC urls (#261) * feat: Have distinct JSON-RPC and gRPC urls * chore: Trigger e2e tests * chore: Fix import groups commit 4e30e27 Author: Bryan White <[email protected]> Date: Wed Dec 13 14:28:36 2023 +0100 fix PR template testing checklist item (#268)
okdas
pushed a commit
that referenced
this pull request
Dec 19, 2023
okdas
pushed a commit
that referenced
this pull request
Nov 14, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
That it says in the title. ☝️
Human Summary
AI Summary
Summary generated by Reviewpad on 13 Dec 23 12:53 UTC
This pull request fixes a checklist item in the PR template related to running E2E tests on DevNet.
Issue
N/A
Type of change
Select one or more:
Testing
make go_develop_and_testmake test_e2e: Add thedevnet-test-e2e` label to the PR. This is VERY expensive, o only do it after all the reviews are complete.Sanity Checklist