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

add VSCode DevContainer #115

Merged
merged 1 commit into from
Sep 5, 2023
Merged

Conversation

vsoch
Copy link
Member

@vsoch vsoch commented Jun 20, 2023

okay - second time is the charm! With @slabasan magic touch, this seemed to work the second time around! This pull request will:

  • Add a VSCode DevContainer to make it easy to develop PerfFlowAspect
  • Add a developer.rst file with instructions for interaction
  • Add an automated build to push the container base to GitHub packages

This will close #112

Once we have the automated build for ghcr.io/flux-framework/perf-flow-aspect-base we can very easily add one for just "ghcr.io/flux-framework/perf-flow-aspect" The use case is wanting a container, ready to go, to develop your application with PerfFlowAspect.

This PR and the subsequent one (I'll open after the container deploys) will subsume #114. With the second container I'll update the developer.rst with another section for how to get started with an example Python app, for example.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
@slabasan
Copy link
Collaborator

Will this resolve #112?

@vsoch
Copy link
Member Author

vsoch commented Jun 21, 2023

yes! Sorry should have put that - let me add to the description.

@slabasan
Copy link
Collaborator

@vsoch I just merged in #114, but now realize this one is meant to replace it. Should I revert #114 in favor of this PR?

@vsoch
Copy link
Member Author

vsoch commented Aug 31, 2023

Yes this was the correct one I think!

@slabasan slabasan merged commit 03160f0 into flux-framework:main Sep 5, 2023
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.

Build trouble in VSCode
2 participants