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.

Sign up for GitHub

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

Jump to bottom

ES6 import for smart contracts tests #10827

Open
clemsos opened this issue Jan 27, 2023 · 2 comments
Open

ES6 import for smart contracts tests #10827

clemsos opened this issue Jan 27, 2023 · 2 comments
Assignees
@clemsos
Labels
dev tools For issues which affect our development process more than the product itself. developer-experience

Comments

@clemsos
Copy link
Member

clemsos commented Jan 27, 2023

The scripts in smart-contracts folder are using some older js patterns such as require and also lack proper linting (at least for the scripts). It will good to give it a refresh and move to ES6 / tsconf to match other folders in the monorepo
@clemsos clemsos added dev tools For issues which affect our development process more than the product itself. developer-experience labels Jan 27, 2023
@clemsos clemsos mentioned this issue Oct 20, 2023
Closed
@clemsos
Copy link
Member Author

clemsos commented Oct 26, 2023

duplicate #12843

@clemsos clemsos closed this as completed Oct 26, 2023
@clemsos clemsos reopened this Mar 5, 2024
@clemsos clemsos changed the title Update smart contracts js tooling ES6 import for smart contracts tests Mar 5, 2024
@clemsos
Copy link
Member Author

clemsos commented Mar 5, 2024

Reopened as this is not solved

@clemsos clemsos mentioned this issue May 21, 2024
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@clemsos clemsos

Labels
dev tools For issues which affect our development process more than the product itself. developer-experience
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@clemsos