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

feat(txwrapper-dev): export WESTEND_SPEC_VERSION constant #313

Merged

Conversation

EmilIvanichkovv
Copy link
Contributor

I propose this change because I intend to utilize the getRegistryWestend function outside this repository. I prefer to rely solely on data from the package, rather than creating a separate constant in my project.

@EmilIvanichkovv EmilIvanichkovv requested a review from a team as a code owner July 5, 2023 10:42
@EmilIvanichkovv
Copy link
Contributor Author

The PR is related to this issue

@TarikGul
Copy link
Member

TarikGul commented Jul 5, 2023

You will need to resolve the conflicts first by pulling in main.

@EmilIvanichkovv EmilIvanichkovv force-pushed the feat/export-WESTEND_SPEC_VERSION branch 4 times, most recently from 1516228 to 2892fd6 Compare July 5, 2023 13:31
Copy link
Collaborator

@marshacb marshacb left a comment

Choose a reason for hiding this comment

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

looks good!

@EmilIvanichkovv EmilIvanichkovv force-pushed the feat/export-WESTEND_SPEC_VERSION branch from 2892fd6 to bbeff23 Compare July 5, 2023 14:16
@EmilIvanichkovv
Copy link
Contributor Author

EmilIvanichkovv commented Jul 5, 2023

I think the tests are a bit flaky. Can you restart them?

@TarikGul
Copy link
Member

TarikGul commented Jul 5, 2023

I think the tests are a bit flaky. Can you restart them?

It's actually a side affect of having multiple metadata files. We are looking for the source of the issue in jest, but it went away when we stopped having multiple metadata sources and stuck with a single source of truth, but recently we realized we need 1 of each chain which brought back the issue to some extent.

@EmilIvanichkovv EmilIvanichkovv force-pushed the feat/export-WESTEND_SPEC_VERSION branch from bbeff23 to 24bf65d Compare July 5, 2023 15:27
@EmilIvanichkovv
Copy link
Contributor Author

Rebased and ready to merge too

@TarikGul TarikGul merged commit 742be6f into paritytech:main Jul 5, 2023
@EmilIvanichkovv EmilIvanichkovv deleted the feat/export-WESTEND_SPEC_VERSION branch July 6, 2023 07:31
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.

3 participants