Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(relayer): only process profitable transactions #408

Merged
merged 15 commits into from
Dec 12, 2022
Merged

feat(relayer): only process profitable transactions #408

merged 15 commits into from
Dec 12, 2022

Conversation

cyberhorsey
Copy link
Contributor

@cyberhorsey cyberhorsey commented Dec 9, 2022 •
edited

closes #247

@codecov
Copy link

codecov bot commented Dec 9, 2022 •
edited

Codecov Report

Merging #408 (6f8ba28) into main (f97db8f) will increase coverage by 0.54%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##             main     #408      +/-   ##
==========================================
+ Coverage   65.02%   65.57%   +0.54%     
==========================================
  Files          96       99       +3     
  Lines        2605     2661      +56     
  Branches      313      313              
==========================================
+ Hits         1694     1745      +51     
- Misses        844      845       +1     
- Partials       67       71       +4
Flag Coverage Δ *Carryforward flag
bridge-ui 100.00% <ø> (ø) Carriedforward from 27cbe5e
protocol 58.54% <ø> (+0.28%) ⬆️ Carriedforward from 27cbe5e
relayer 69.14% <77.77%> (+0.80%) ⬆️
ui 100.00% <ø> (ø) Carriedforward from 27cbe5e

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
packages/relayer/repo/event.go 72.72% <55.55%> (-2.28%) ⬇️
packages/relayer/cli/cli.go 61.50% <60.00%> (+0.32%) ⬆️
packages/relayer/message/process_message.go 61.79% <62.50%> (-5.75%) ⬇️
packages/relayer/message/get_latest_nonce.go 72.72% <72.72%> (ø)
packages/relayer/message/is_profitable.go 76.00% <76.00%> (ø)
packages/relayer/message/wait_for_confirmations.go 80.00% <80.00%> (ø)
packages/relayer/indexer/filter_then_subscribe.go 23.52% <85.71%> (+3.77%) ⬆️
packages/relayer/http/get_events_by_address.go 88.00% <100.00%> (+38.00%) ⬆️
packages/relayer/indexer/service.go 75.26% <100.00%> (+0.26%) ⬆️
packages/relayer/message/processor.go 100.00% <100.00%> (ø)
... and 3 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dantaik dantaik enabled auto-merge (squash) December 10, 2022 04:01
@dantaik dantaik merged commit b5d8180 into main Dec 12, 2022
@dantaik dantaik deleted the profitable_tx branch December 12, 2022 15:43
1xDeFi added a commit to 1xDeFi/taiko-mono that referenced this pull request Dec 13, 2022
@1xDeFi
Squashed commit of the following:
c511313 
commit 1a94184
Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
Date:   Tue Dec 13 16:56:11 2022 +0100

    Squashed commit of the following:

    commit a8b1c49
    Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
    Date:   Mon Dec 12 14:15:36 2022 +0100

        remove internal variables and functions

        Removed from: https://taiko.xyz/docs/reference/smart-contracts/bridge/BridgedERC20

commit 235b685
Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
Date:   Tue Dec 13 16:56:00 2022 +0100

    Squashed commit of the following:

    commit 1fdf217
    Merge: 4ae3dbc 788e162
    Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
    Date:   Tue Dec 13 16:24:24 2022 +0100

        Merge branch 'main' into patch-5

    commit 4ae3dbc
    Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
    Date:   Mon Dec 12 14:14:20 2022 +0100

        remove internal variables and functions

        Removed from: https://taiko.xyz/docs/reference/smart-contracts/bridge/

commit 1a28290
Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
Date:   Tue Dec 13 16:55:46 2022 +0100

    Squashed commit of the following:

    commit 5bdd441
    Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
    Date:   Mon Dec 12 14:10:58 2022 +0100

        remove internal variables and functions

        Removed for: https://taiko.xyz/docs/reference/smart-contracts/L1/TaikoL1

commit 9e219bd
Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
Date:   Tue Dec 13 16:55:23 2022 +0100

    Squashed commit of the following:

    commit 19525cb
    Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
    Date:   Mon Dec 12 14:08:55 2022 +0100

        remove internal variables and functions

        Removed from: https://taiko.xyz/docs/reference/smart-contracts/L1/TkoToken

commit e443de4
Merge: bc54a67 788e162
Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
Date:   Tue Dec 13 16:26:26 2022 +0100

    Merge branch 'taikoxyz:main' into patch-2

commit 788e162
Author: Daniel Wang <99078276+dantaik@users.noreply.github.com>
Date:   Tue Dec 13 06:44:16 2022 +0800

    chore(website): publish website when contract code changes on main (taikoxyz#413)

commit b5d8180
Author: jeff <113397187+cyberhorsey@users.noreply.github.com>
Date:   Mon Dec 12 07:43:45 2022 -0800

    feat(relayer): only process profitable transactions (taikoxyz#408)

commit bc54a67
Author: 1xDeFi <90858998+1xDeFi@users.noreply.github.com>
Date:   Mon Dec 12 14:07:20 2022 +0100

    remove internal variables and functions

    Removed from: https://taiko.xyz/docs/reference/smart-contracts/L2/TaikoL2
@github-actions github-actions bot mentioned this pull request Jan 18, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dantaik dantaik dantaik approved these changes

@davidtaikocha davidtaikocha davidtaikocha approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[relayer] only process profitable transactions
3 participants
@cyberhorsey @dantaik @davidtaikocha