Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Added build on launch#2

Open
MiniFalafel wants to merge 11 commits intopeter1745:mainfrom
MiniFalafel:main
Open

Added build on launch#2
MiniFalafel wants to merge 11 commits intopeter1745:mainfrom
MiniFalafel:main

Conversation

@MiniFalafel
Copy link

I'm not an expert on how everyone else organizes their projects and workspaces, so I'm not 100% positive this will work for every code base. Here's a summary of the changes:

  1. launch.json and tasks.json contents now generate into .code-workspace file.
    • Lets you run the build command from the workspace root directory, independent of where project folders are (sub-directories, sub-sub-directories, etc.)
    • Launch configs generate based on the startproject specified by the user in their premake script.
  2. Moved launch and task generation code into their own files to let vscode_workspaces access both.
  3. Launch configs generate with a "preLaunchTask": that calls respective build task.
    • Might want to also generate launch configs that don't build in case user wants to run without re-building.

@peter1745
Copy link
Owner

Hey! I don't really actively maintain this project anymore due to me moving away from VS Code, however if you're finding this useful and is willing to maintain it I'd be happy to make your fork the main repo if that's alright with you 🙂

@MiniFalafel
Copy link
Author

Yeah man! I'd be happy to if that works better for you:)

@MiniFalafel MiniFalafel reopened this Sep 30, 2024
@peter1745
Copy link
Owner

Yeah man! I'd be happy to if that works better for you:)

Awesome, I'll archive this repo and update the readme to link to your fork of it 🙂

Added reference to original project + general updates
Comments help me understand the existing code better. Also better for future maintenance.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants