Skip to content

itest improvements (docs references and vscode examples) #990

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ZZiigguurraatt
Copy link

@ZZiigguurraatt ZZiigguurraatt commented Feb 27, 2025

Cherry picked changes from #898 . Also working to add some documentation and other improvements

  • Rename .vscode to .vscode-samples
  • Provide an example to run itests in vscode
  • Add a minimal itest README file

jharveyb and others added 4 commits February 27, 2025 12:23
This commit changes the mode of itest configuration in `launch.json` to
test-mode. It also removes the unused flag `btcdexec`. The binary is
expected by lnd though, so it still needs to be built.
@ZZiigguurraatt ZZiigguurraatt force-pushed the general_itest_improvements branch from 9c464a2 to bb21c3c Compare February 27, 2025 17:25
@ZZiigguurraatt ZZiigguurraatt changed the title General itest improvements General itest improvements and vscode examples Feb 27, 2025
@ZZiigguurraatt
Copy link
Author

ZZiigguurraatt commented Feb 27, 2025

@gijswijs, would appreciate if you could take a look and compare to ZZiigguurraatt#1. github is not allowing me to add you as a reviewer for some reason.

@ZZiigguurraatt
Copy link
Author

ZZiigguurraatt commented Feb 27, 2025

Does it make sense to use litd_no_ui in the buildFlags for itest? Not sure if the GUI is needed for running itests?

Or, maybe the Makefile needs to be modified to have an option that does not include build-itest: app-build?

@ZZiigguurraatt ZZiigguurraatt changed the title General itest improvements and vscode examples itest improvements (docs references and vscode examples) Feb 27, 2025
@ZZiigguurraatt ZZiigguurraatt marked this pull request as ready for review February 28, 2025 16:06
@levmi levmi moved this from 🆕 New to 🏗 In progress in Taproot-Assets Project Board Mar 3, 2025
@levmi levmi requested a review from gijswijs March 3, 2025 16:36
@ZZiigguurraatt ZZiigguurraatt force-pushed the general_itest_improvements branch from b408d20 to c0e0097 Compare March 13, 2025 15:32
@lightninglabs-deploy
Copy link

@gijswijs: review reminder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

4 participants