Skip to content

feat: bridge-indexer integration#286

Merged
karim-en merged 18 commits intomainfrom
feat/bridge-indexer
May 9, 2025
Merged

feat: bridge-indexer integration#286
karim-en merged 18 commits intomainfrom
feat/bridge-indexer

Conversation

@frolvanya
Copy link
Copy Markdown
Contributor

@frolvanya frolvanya commented Mar 4, 2025

No description provided.

@frolvanya frolvanya force-pushed the feat/bridge-indexer branch 2 times, most recently from 06d2f37 to 6f8da1b Compare March 5, 2025 03:02
@frolvanya frolvanya force-pushed the feat/bridge-indexer branch from 64450c9 to 9a08efb Compare March 16, 2025 22:48
@frolvanya frolvanya marked this pull request as ready for review March 16, 2025 22:50
@frolvanya
Copy link
Copy Markdown
Contributor Author

Since now we have a dependency from bridge-indexer-types we need to fix our CI, since this crate is in private repo. Maybe this can be done by adding some access keys to GitHub secrets

@frolvanya frolvanya changed the title feat: migration to bridge-indexer feat: bridge-indexer integration Mar 16, 2025
Comment thread omni-relayer/src/startup/bridge_indexer.rs
Comment thread omni-relayer/src/startup/bridge_indexer.rs
Comment thread omni-relayer/src/startup/bridge_indexer.rs Outdated
Comment thread omni-relayer/src/startup/bridge_indexer.rs Outdated
Comment thread omni-relayer/src/startup/bridge_indexer.rs Outdated
@frolvanya frolvanya force-pushed the feat/bridge-indexer branch 7 times, most recently from 2d2a26b to d147f9c Compare April 12, 2025 22:47
@frolvanya frolvanya changed the base branch from main to feat/retry-fee April 12, 2025 23:23
@frolvanya frolvanya changed the base branch from feat/retry-fee to main April 12, 2025 23:23
@frolvanya frolvanya force-pushed the feat/bridge-indexer branch from 2cbc7ea to dd92de3 Compare April 12, 2025 23:24
@frolvanya frolvanya requested review from Copilot and olga24912 April 12, 2025 23:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 25 out of 26 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • omni-relayer/.example-env: Language not supported
Comments suppressed due to low confidence (3)

omni-relayer/src/utils/evm.rs:237

  • [nitpick] If possible, consider converting tokenAddress to snake_case (e.g. token_address) for consistency with Rust naming conventions, unless this naming is dictated by an external schema.
tokenAddress: init_log.inner.tokenAddress,

omni-relayer/src/utils/mod.rs:2

  • Removing the fee module might break functionality if there are still references to it. Please verify that no other parts of the codebase require the fee module or that appropriate updates have been made.
-pub mod fee;

.github/workflows/release.yml:62

  • [nitpick] Multiple SSH keys are concatenated in this input. Ensure that only the intended key is provided or that the parsing logic correctly handles multiple keys to avoid authentication issues during deployment.
ssh-private-key: |

@frolvanya
Copy link
Copy Markdown
Contributor Author

frolvanya commented Apr 13, 2025

Well, looks like copilot is still not able to check if modules/files exist before writing a comment 😅

@frolvanya frolvanya force-pushed the feat/bridge-indexer branch from e92dfe1 to 350ac25 Compare April 24, 2025 02:21
@frolvanya frolvanya force-pushed the feat/bridge-indexer branch from 56494de to 53550b7 Compare April 24, 2025 02:27
@karim-en karim-en merged commit f7b84ed into main May 9, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[OmniBridge]: refactor relayer to use bridge-indexer as a data source for incoming events

4 participants