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

Traditional non docker environment for backend development #1364

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jerryfletcher21
Copy link
Contributor

@jerryfletcher21 jerryfletcher21 commented Jul 4, 2024

What does this PR do?

Builds on #1363.
This PR adds a setup for a traditional non docker regtest environment for backed development and testing.
scripts/traditional/README.md with all the informations.

Checklist before merging

  • Install pre-commit and initialize it: pip install pre-commit, then pre-commit install. Pre-commit installs git hooks that automatically check the codebase. If pre-commit fails when you commit your changes, please fix the problems it points out.

@Reckless-Satoshi
Copy link
Collaborator

Reckless-Satoshi commented Jul 7, 2024

This is super useful! What about we base the tradition-environment over main instead of over split-host-port given how traumatic that fix is?

@jerryfletcher21
Copy link
Contributor Author

This is super useful! What about we base the tradition-environment over main instead of over split-host-port given how traumatic that fix is?

It would be a bit less clean since we would have to extract the port from the variables, but it is of course straightforward to do.

@jerryfletcher21 jerryfletcher21 force-pushed the traditional-environment branch from 1532728 to f9d591a Compare July 11, 2024 16:04
@jerryfletcher21 jerryfletcher21 force-pushed the traditional-environment branch from f9d591a to 76ad7e3 Compare July 13, 2024 13:05
@Reckless-Satoshi Reckless-Satoshi force-pushed the main branch 2 times, most recently from 9054b87 to f9244e1 Compare September 10, 2024 01:28
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.

2 participants