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.

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

[release] 1.0.3-beta #1761

Closed
wants to merge 96 commits into from
Closed

[release] 1.0.3-beta #1761

wants to merge 96 commits into from

Conversation

ca333
Copy link

@ca333 ca333 commented Apr 21, 2023

No description provided.

rozhkovdmitrii and others added 30 commits March 22, 2023 11:48
…-adex

# Conflicts:
#	Cargo.lock
#	mm2src/common/Cargo.toml
* improve CI/CD logic

Signed-off-by: ozkanonur <work@onurozkan.dev>

* add changelog entry

Signed-off-by: ozkanonur <work@onurozkan.dev>

---------

Signed-off-by: ozkanonur <work@onurozkan.dev>
Reviewed-by: shamardy <shamardy@yahoo.com>
rozhkovdmitrii and others added 18 commits April 6, 2023 18:05
* stream stdout/err to a file

Signed-off-by: ozkanonur <work@onurozkan.dev>

* add changelog entry

Signed-off-by: ozkanonur <work@onurozkan.dev>

* use default target for `lightning::util::Logger`

Signed-off-by: ozkanonur <work@onurozkan.dev>

* add new config parameter `silent_console`

Signed-off-by: ozkanonur <work@onurozkan.dev>

* ignore mergetool temp files

Signed-off-by: ozkanonur <work@onurozkan.dev>

* sync back the `Cargo.lock`

Signed-off-by: ozkanonur <work@onurozkan.dev>

---------

Signed-off-by: ozkanonur <work@onurozkan.dev>
Reviewed-by: cipig, shamardy <shamardy@yahoo.com>
Signed-off-by: ozkanonur <work@onurozkan.dev>
Reviewed by: shamardy <shamardy@yahoo.com>
Return latest nonce quickly without waiting for all nodes to sync.

Send evm transactions to only the nodes that synced the latest nonce.

---------

Reviewed-by: laruh, caglaryucekaya <caglaryucekaya@gmail.com>, ozkanonur <work@onurozkan.dev>
@shamardy
Copy link
Collaborator

We seem to have already merged code to main appear again in this PR because of how 1.0.2-beta commit was merged by squashing, I will look into how to fix this today to have a consistent git history for both dev and main branch.

@shamardy
Copy link
Collaborator

shamardy commented Apr 21, 2023

To fix the issue I mentioned in #1761 (comment):

  1. I will reset the dev branch to the state of the main branch locally using git reset --hard main
  2. I will merge these 3 pull requests branches locally remove unnecessary files #1754, update compilation profiles #1759, avoid waiting for all EVM nodes to sync the latest nonce #1757.
  3. Then I will force push the changes to the dev branch by using git push -f origin dev

What everybody thinks @KomodoPlatform/mm2?

@shamardy
Copy link
Collaborator

shamardy commented Apr 21, 2023

@ca333 @ozkanonur most agreed in DM to a second solution where:

  1. We don't force push and instead merge main to dev to fix conflicts and include the v1.0.2-beta commit in dev
  2. When merging this PR to main we don't squash the commits.

onur-ozkan and others added 3 commits April 22, 2023 19:11
* optimize p2p network layer

Signed-off-by: ozkanonur <work@onurozkan.dev>

* update default gas limit for tendermint

Signed-off-by: ozkanonur <work@onurozkan.dev>

* rollback topic limit

Signed-off-by: ozkanonur <work@onurozkan.dev>

* remove unused arg from `lp_ordermatch::process_msg`

Signed-off-by: ozkanonur <work@onurozkan.dev>

* use OrderbookP2PHandlerResult for process_orders_keep_alive and process_maker_order_updated

* return one error for failed SwapMsg, SwapStatus deserialization

* fix process_swap_msg in wasm to return error if SwapMsg can't be deserialized

* remove additional space in SwapMsg, SwapStatus decode error message

* roll back crate visibility for new_protocol mod

---------

Signed-off-by: ozkanonur <work@onurozkan.dev>
Co-authored-by: shamardy <shamardy@yahoo.com>
Signed-off-by: ozkanonur <work@onurozkan.dev>
Signed-off-by: ozkanonur <work@onurozkan.dev>
Reviewed-by: cipig, shamardy <shamardy@yahoo.com>
@onur-ozkan onur-ozkan added the blocked requires additional attention label Apr 25, 2023
@ca333
Copy link
Author

ca333 commented Apr 26, 2023

closing this due mistakenly propagating the conflict resolution to a new branch - will push directly to dev instead

@ca333 ca333 closed this Apr 26, 2023
@ca333 ca333 deleted the 1.0.3-conflict-resolution branch April 26, 2023 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked requires additional attention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants