You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Copy file name to clipboardExpand all lines: PULL_REQUEST_TEMPLATE.md
+3-6Lines changed: 3 additions & 6 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -3,20 +3,17 @@ If so, link that issue(s) here.
3
3
4
4
**Fill out the following table describing your edits:**
5
5
6
-
| Original Query | Updated Query| Change | Reasoning |
6
+
| Original | Updated | Change | Reasoning |
7
7
|---|---|---|---|
8
8
|[3237745](https://dune.com/queries/3237745)|[3237938](https://dune.com/queries/3238935)| Remove sandwich traders using dex.sandwiches | We should only care about traders who are not doing MEV |
|`evm_grants.csv`|n/a|added grants for Optimism | captures approved grants only|
10
10
11
11
**Provide any other context or screenshots that explain or justify the changes above:**
12
12
13
-
"`tx_from` may not be the most accurate, we might want to use `tx_to` instead since that identifies the MEV contract being utilized. But this requires further digging"
14
-
15
13
---
16
14
17
15
**Note to Contributor:**
18
16
19
17
Make sure your PR edits the original `query_id.sql` file with the new query text. If you are proposing adding a new query completely, make sure to add it to `queries.yml` and as a file in `/queries` as well.
0 commit comments