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: better scripts and test automation #200

Merged
merged 31 commits into from
Mar 14, 2024

Conversation

0xneves
Copy link
Contributor

@0xneves 0xneves commented Mar 14, 2024

Feature Request

Describe Feature

This is a major refactor in the scripts of Swaplace to automate the tests in general.
Based on contributors' feedback, we realize there is a learning curve for developers who are not familiar with smart contracts, to prepare their own set of contracts in a localnet or testnet.

Describe Preferred Solution

  • Create a Makefile with automation for the entire flux
  • Create scripts that can be called in a standalone way without the need to edit parameters and configurations
  • Add typings and better errors for every single one of them
  • Add an easy-to-use transaction filler to aid in the Ponder App development
  • Add dynamic variables in .env to store the configurations automatically

0xneves and others added 30 commits February 20, 2024 12:46
- it will sabe directly in the .env
- was causing errors where the instance was not saving the replacements correctly after the script finished
- following industry standards
@0xneves 0xneves added bug Something isn't working enhancement New feature or request labels Mar 14, 2024
@0xneves 0xneves self-assigned this Mar 14, 2024
@0xneves 0xneves changed the base branch from main to develop March 14, 2024 13:29
@0xneves 0xneves merged commit 5ef5aa3 into blockful-io:develop Mar 14, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants