Skip to content
This repository has been archived by the owner on Dec 5, 2021. It is now read-only.

Move wallet repo and development to monorepo #35

Merged
merged 33 commits into from
Jun 8, 2021

Conversation

CAPtheorem
Copy link

@CAPtheorem CAPtheorem commented Jun 1, 2021

*Prepare to EOL/Archive omgx_contracts
*Move dockers/envs to /ops - where possible
*Remove local dockers/envs from /packages/omgx/message-relayer-fast
*Re-enable docker-compose up --build --detach
*Confirm correct spin up of the optimism stack
*Remove message-relayer-fast
*Confirm unit/integration tests for the wallet stack
*Prepare for alternative approach (white_listing) for fast message relay

…er-fast; and connect up message-relayer-fast with the Dockers/ops
@codecov-commenter
Copy link

codecov-commenter commented Jun 1, 2021

Codecov Report

Merging #35 (6757064) into develop (060ef21) will increase coverage by 1.53%.
The diff coverage is 25.00%.

❗ Current head 6757064 differs from pull request most recent head 1d33a94. Consider uploading reports for the commit 1d33a94 to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #35      +/-   ##
===========================================
+ Coverage    80.60%   82.14%   +1.53%     
===========================================
  Files           50       49       -1     
  Lines         1939     1904      -35     
  Branches       311      304       -7     
===========================================
+ Hits          1563     1564       +1     
+ Misses         376      340      -36     
Impacted Files Coverage Δ
...VM/bridge/messaging/OVM_L1CrossDomainMessenger.sol 92.98% <25.00%> (-5.14%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 060ef21...1d33a94. Read the comment docs.

@CAPtheorem CAPtheorem changed the title Merge omgx_contracts into /optimism monorepo; integrate message-relay… Move wallet in to monorepo Jun 3, 2021
@CAPtheorem CAPtheorem changed the title Move wallet in to monorepo Move wallet repo and development to monorepo Jun 3, 2021
@github-actions github-actions bot added the A-ci label Jun 4, 2021
@CAPtheorem CAPtheorem requested a review from InoMurko June 7, 2021 18:41
Copy link

@InoMurko InoMurko left a comment

Choose a reason for hiding this comment

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

I'd like to take another look but I do think we should revert all lint changes, these are gonna cause unnecessary headaches later on

Copy link

@souradeep-das souradeep-das left a comment

Choose a reason for hiding this comment

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

yes, let's try to revert lint changes to upstream, will be helpful!

@CAPtheorem CAPtheorem merged commit 4a7e3cf into develop Jun 8, 2021
InoMurko pushed a commit that referenced this pull request Jul 12, 2021
fix: move all /home/vault to /vault
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants