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

[NTRN-127] chore: small improvements #94

Merged
merged 5 commits into from
Nov 1, 2023

Conversation

sotnikov-s
Copy link
Contributor

@sotnikov-s sotnikov-s commented Oct 21, 2023

task: https://hadronlabs.atlassian.net/browse/NTRN-127

this PR:

  • adds missing set_contract_version for the reserve and distribution contracts;
  • optimises migration params check in execute_migrate_from_xyk_to_cl.

D.O.D. 6 from the task has been skipped due to majority is opposed to it.

@sotnikov-s
Copy link
Contributor Author

@pr0n00gler
Copy link
Collaborator

pr0n00gler commented Oct 21, 2023

https://github.com/neutron-org/neutron-tests/actions/runs/6589360521

You use main of the integration-tests repo in your test run, but you should use feat/xyk-to-cl-migration since the PR is related to the migration of liquidity

@sotnikov-s
Copy link
Contributor Author

test run against feat/xyk-to-cl-migration branch:
https://github.com/neutron-org/neutron-tests/actions/runs/6614904903

test run against feat/lockdrop-cl-pools-migration branch:
https://github.com/neutron-org/neutron-tests/actions/runs/6612057482

Copy link
Collaborator

@pr0n00gler pr0n00gler left a comment

Choose a reason for hiding this comment

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

It's out of scope, but since you've updated the reserve contract, please add similar CONTRACT_NAME and CONTRACT_VERSION constants into the distribution contract (and cw2 methods for setting contract info)

contracts/tokenomics/reserve/src/contract.rs Outdated Show resolved Hide resolved
@sotnikov-s
Copy link
Contributor Author

test run against feat/xyk-to-cl-migration branch:
https://github.com/neutron-org/neutron-tests/actions/runs/6637554276

test run against feat/lockdrop-cl-pools-migration branch:
https://github.com/neutron-org/neutron-tests/actions/runs/6637560757

@sotnikov-s sotnikov-s requested a review from pr0n00gler October 25, 2023 10:31
@sotnikov-s
Copy link
Contributor Author

reran the above jobs:

test run against feat/xyk-to-cl-migration branch:
https://github.com/neutron-org/neutron-tests/actions/runs/6637554276

test run against feat/lockdrop-cl-pools-migration branch:
https://github.com/neutron-org/neutron-tests/actions/runs/6637560757

@sotnikov-s sotnikov-s requested a review from pr0n00gler October 26, 2023 06:16
Copy link
Contributor

@NeverHappened NeverHappened left a comment

Choose a reason for hiding this comment

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

Looks nice!

@pr0n00gler pr0n00gler requested a review from swelf19 October 31, 2023 14:35
@pr0n00gler pr0n00gler merged commit 55c9a83 into feat/pcl-pools Nov 1, 2023
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.

4 participants