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 basic Python devcontainer configuration. #6

Merged
merged 1 commit into from
Dec 27, 2023

Conversation

alunduil
Copy link
Contributor

This sets up a configuration for VS Code devcontainers. Which allows anyone with VS Code using the Dev Containers plugin to have a prescribed development environment ready to go immediately with all needed dependencies. This configuration may be incomplete as it's just the initial configuration.

Copy link
Owner

@Flameeyes Flameeyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll need to add SPDX headers on it for pre-commit to pass.

@alunduil alunduil force-pushed the vscode/devcontainer branch from e6c9aed to 58ca7f1 Compare December 27, 2023 10:12
@alunduil
Copy link
Contributor Author

You'll need to add SPDX headers on it for pre-commit to pass.

Do you recommend doing that as a header comment or an extra file like I see you doing for your data files?

This sets up a configuration for VS Code devcontainers.  Which allows anyone
with VS Code using the Dev Containers plugin to have a prescribed development
environment ready to go immediately with all needed dependencies.  This
configuration may be incomplete as it's just the initial configuration.
@alunduil alunduil force-pushed the vscode/devcontainer branch from 58ca7f1 to 865ae44 Compare December 27, 2023 14:52
@alunduil alunduil requested a review from Flameeyes December 27, 2023 14:52
@Flameeyes Flameeyes merged commit ba6337a into Flameeyes:main Dec 27, 2023
1 check failed
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