vscode-dev-containers/CONTRIBUTING.md

268 строки
19 KiB
Markdown
Исходник Обычный вид История

2019-03-27 22:22:15 +03:00
# Contributing
2019-04-30 02:32:26 +03:00
This document outlines a number of ways you can get involved.
2019-03-27 22:22:15 +03:00
2019-04-11 02:04:21 +03:00
## Contributing Dev Container Definitions
2019-05-18 00:22:14 +03:00
This repository contains a set of **dev container definitions** to help get you up and running with a containerized environment. The definitions describe the appropriate container image, runtime arguments for starting the container, and VS Code extensions that should be installed. They are intended to be dropped into an existing project or folder rather than acting as sample projects. (See the [vscode-remote-try-*](https://github.com/search?q=org%3Amicrosoft+vscode-remote-try-&type=Repositories) repositories if you are looking for sample projects.)
Have a container set up you're proud of and would like to share? Want to see some changes made to an existing definition? We love contributions! Read on to learn how.
2019-06-07 02:27:57 +03:00
### What makes a good Dev Container Definition?
2019-06-07 02:31:37 +03:00
A good definition in this repository will:
2019-06-07 02:27:57 +03:00
2019-06-07 02:31:37 +03:00
- Solve a common setup or installation challenge
- Illustrate a unique container runtime configuration
- Highlight an important tip or trick
2020-01-18 12:50:22 +03:00
- Or provide an easy to understand template for developers that are just getting started with dev containers.
2020-01-08 19:34:45 +03:00
- Works on Linux, macOS, and Windows
2019-06-07 02:31:37 +03:00
Since devcontainer.json can be commited to a source code repository, the definitions here are not intended to cover every possible permutation. When thinking about contributing a new dev container definition, consider the following questions:
2019-06-07 02:27:57 +03:00
1. How different is the scenario you are interested in from other dev container definitions in the repository?
- Does it drive significantly different extension, runtime, configuration requirements?
2020-06-02 01:47:13 +03:00
- Are the requirements cumbersome to install or are they typically things added by a package manager (pip, npm, etc) using a project's manifest file (`package.json`, `requirements.txt`, etc)?
2. How likely are other developers to find the definition useful on its own? Could the scenario be broadened to help more people?
2019-06-07 02:27:57 +03:00
2020-01-18 12:50:22 +03:00
If the definition is too similar to others, consider contributing a PR to improve an existing one instead. If the scenario is too specific consider generalizing it and making it more broadly applicable.
2019-06-07 02:27:57 +03:00
To help speed up PRs, we encourage you to install the recommended [EditorConfig extension](https://marketplace.visualstudio.com/items?itemName=EditorConfig.EditorConfig). This will ensure your changes adhere to our style guidelines.
### A note on referenced images and Dockefile contents
One of the things we want to be sure anyone using a definition from this repository is able to do is understand what is inside it. In general, images referenced by Dockerfiles, Docker Compose files, or devcontainer.json in this repository should reference known base Docker, Microsoft, or runtime/platform community/vendor managed base images. These images are already well maintained, regularly patched, and maintained by the platform/runtime community or vendor. From there you can use a Dockerfile to add any additional contents.
2021-05-28 17:41:27 +03:00
When adding contents to the Dockerfile, use official sources. There are a number of pre-built images you can take advantage of to speed things up for yourself and other developers to reduce build times. In particular, take note of images like `mcr.microsoft.com/vscode/devcontainers/base:debian` and `mcr.microsoft.com/vscode/devcontainers/base:ubuntu` ([see here for a list](https://hub.docker.com/_/microsoft-vscode-devcontainers)). These are like the base `debian` and `ubuntu` images but include things developers commonly need or want (e.g. git and curl).
If you are using something like `curl` or `wget` to download the contents, add a checksum verification if one is available. Using package managers like `apt-get` on Debian/Ubuntu with 3rd party sources can also be a way to do this verification since they require adding the source's signing key.
Note that other definitions in this repository use Debian or Ubuntu bases in the vast majority of cases. We recommend using this as the variation of the base image you use wherever possible so you can avoid questions from developers new to working with Linux.
### Expectations for new definitions
When contributing a new definition, be sure you are also willing to sign up to maintain the definition over time. The likelihood of other maintainers in this repository having the needed knowledge properly maintain your definition is low, so you'll be contacted about PRs and issues raised about them. Please include your GitHub alias (and if desired other contact info) in the README so that you are identified as the maintainer of the definition. You will "@'d" in on PRs and issues raised pertaining to the definition.
Over time, if you no longer want to maintain the definition and do not have a suitable replacement, we can remove it from the repository. Just raise an issue or PR and let us know. This repository's maintainers may also need to do this proactively if an issue is raised that is breaking and we are unable to get a timely response in resolving the issue.
2019-05-18 00:22:14 +03:00
### Anatomy of a Dev Container Definition
2020-06-02 01:47:13 +03:00
The contents of the folders in the `containers` directory ultimately populate the available definitions list shown in the **Remote-Containers: Add Development Container Configuration Files...** command. To make this work, each folder consists of up to three elements:
2019-05-18 00:22:14 +03:00
1. **The container definition itself** - These are the files and folders that will be added to a user's existing project / folder if they select the definition. Typically these files are stored in a `.devcontainer` folder.
2. **Test assets** - While you are creating your definition, you may need to use a test project to make sure it works as expected. Contributing these files back will also help others that want to contribute to your definition in the future. These files are typically located in a `test-project` folder.
3. **A `.npmignore` file** - This tells VS Code which files in the folder should be ignored when a user selects it for their project / folder. The file typically lists test assets or folders.
2019-05-18 00:22:14 +03:00
### Creating a new definition
Most contributions require you to agree to a Contributor License Agreement (CLA) declaring that you have the right to, and actually do, grant us the rights to use your contribution. For details, visit https://cla.microsoft.com.
2019-04-11 02:04:21 +03:00
When you submit a pull request, a CLA-bot will automatically determine whether you need to provide a CLA and decorate the PR appropriately (e.g., label, comment). Simply follow the instructions provided by the bot. You will only need to do this once across all repos using our CLA.
2019-05-18 00:22:14 +03:00
To create a new definition:
2019-04-11 02:04:21 +03:00
1. Fork and clone this repository
2. Create a new folder in the `containers` directory. The name of the folder is effectively the **definition ID** and should follow the following format:
````text
<language>-<optional: version>-<descriptor>
````
You'll find many examples in the current `containers` folder.
3. You can grab one of the templates from the `container-templates` folder to help you get an idea of what to contribute for different scenarios, but here's a quick summary of what you should include:
```text
📁 <language>-<optional: version>-<descriptor>
📁 .devcontainer
📄 devcontainer.json
📄 Dockerfile (optional)
📄 docker-compose.yml (optional)
📁 test-project (optional)
📄 .npmignore
📄 README.md
```
See the [VS Code Remote Development documentation](https://aka.ms/vscode-remote/docker) for information on the expected contents of `devcontainer.json` and how it relates to other files listed above.
2020-06-02 01:47:13 +03:00
Note that any additional assets can be included as needed, but keep in mind that these will overlay on top of an existing project. Keeping these files in the `.devcontainer` folder should reduce the chances of something conflicting but note that any commands that are run are relative to the root of the project, so you'll need to include `.devcontainer` in any path references.
2019-04-11 02:04:21 +03:00
Finally, create a `README.md` in the folder with a brief description of the purpose of the container definition and any manual steps required to use it.
2019-05-18 00:22:14 +03:00
4. Update [`.npmignore`](https://docs.npmjs.com/misc/developers#keeping-files-out-of-your-package) if you've added new folders that should be excluded if used. Add anything you don't want copied in to a user's existing project / folder into this file in [glob](https://facelessuser.github.io/wcmatch/glob/) form.
2019-04-11 02:04:21 +03:00
2020-07-15 18:57:17 +03:00
### Why do Dockerfiles in this repository use RUN statements with commands separated by &&?
2020-07-15 18:57:17 +03:00
Each `RUN` statement creates a Docker image "layer". If one `RUN` statement adds in temporary contents, these contents remain in this layer in the image even if they are deleted in a subsequent `RUN`. This means the image takes more storage locally and results in slower image download times if you publish the image to a registry.
So, in short, you want to clean up after you install or configure anything in the same `RUN` statement. To do this, you can either:
1. Use a string of commands that cleans up at the end. e.g.:
```Dockerfile
RUN apt-get update && apt-get -y install --no-install-recommends git && apt-get autoremove -y && apt-get clean -y && rm -rf /var/lib/apt/lists/*
```
... or across multiple lines (note the `\` at the end escaping the newline):
```Dockerfile
RUN apt-get update \
&& apt-get -y install git \
&& apt-get autoremove -y \
&& apt-get clean -y \
&& rm -rf /var/lib/apt/lists/*
```
2. Put the commands in a script, temporarily copy it into the container, then remove it. e.g.:
```Dockerfile
COPY ./my-script.sh /tmp/my-script.sh
RUN bash /tmp/my-script.sh \
&& rm -f /tmp/my-script.sh
```
Some other tips:
2021-05-28 17:41:27 +03:00
1. You'd be surprised [how big package lists](https://askubuntu.com/questions/179955/var-lib-apt-lists-is-huge) can get, so be sure to clean these up too. Most Docker images that use Debian / Ubuntu use the following command to clean these up:
```
rm -rf /var/lib/apt/lists/*
```
2020-07-23 19:00:37 +03:00
The only downside of doing this is that `apt-get update` has to be executed before you install a packages. However, in most cases adding this package to a Dockerfile is a better choice anyway since this will survive a "rebuild" of the image and the creation of an updated container.
2. Use the scripts in the [script library](./script-library) in this repository where appropriate. You do not even need to copy the script into your `.devcontainer` folder to use it. See the [README](./script-library) for details. Most existing definitions use the "common" script to ensure things like `git`, a non-root user, and useful command line utilities like `ps`, `ip`, `jq` are present.
3. In all cases, you'll want to pay attention to package caching since this can also take up image space. Typically there is an option for a package manager to not cache when installing that you can use to minimize the size of the image. For example, for Alpine Linux, there's `apk --no-cache`
2020-07-23 19:00:37 +03:00
4. Watch out for the installation of "recommended" packages you don't need. By default, Debian / Ubuntu's `apt-get` installs packages that are commonly used with the one you specified - which in many cases isn't required. You can use `apt-get -y install --no-install-recommends` to avoid this problem.
2019-04-11 02:04:21 +03:00
### Developing and testing a definition
2020-06-02 01:47:13 +03:00
VS Code Remote provides a straightforward development loop for creating and editing container definitions. Just follow these steps to get started:
2019-04-11 02:04:21 +03:00
1. Create a definition folder and open it in VS Code
2. Edit the contents of the definition
2020-01-18 12:50:22 +03:00
3. Try it with <kbd>F1</kbd> > **Remote-Containers: Reopen Folder in Container**.
2019-05-18 00:22:14 +03:00
4. On failure:
2020-01-18 12:50:22 +03:00
1. <kbd>F1</kbd> > **Remote-Containers: Reopen Folder Locally**, which will open a new local window.
2019-05-18 00:22:14 +03:00
2. In this local window: Edit the contents of the `.devcontainer` folder as required.
2020-01-18 12:50:22 +03:00
3. Try it again: Go back to the container window, <kbd>F1</kbd> > **Developer: Reload Window**.
2019-05-18 00:22:14 +03:00
4. Repeat as needed.
5. If the build was successful, but you want to make more changes:
1. Edit the contents of the `.devcontainer` folder as required when connected to the container.
2020-01-18 12:50:22 +03:00
2. <kbd>F1</kbd> > **Remote-Containers: Rebuild Container**.
2019-05-18 00:22:14 +03:00
3. On failure: Follow the same workflow above.
2020-06-02 01:47:13 +03:00
Note that if you make major changes, Docker may occasionally not pick up your edits. If this happens, you can delete the existing container and image, open the folder locally, and go to step 2 above. Install the [Docker extension](https://marketplace.visualstudio.com/items?itemName=PeterJausovec.vscode-docker) locally (when not in a container) to make this easy.
2019-04-11 02:04:21 +03:00
2019-05-18 00:22:14 +03:00
After you get your container up and running, you can test it by adding test assets / projects into the definition folder and then adding their locations to the `.npmignore` file in [glob](https://facelessuser.github.io/wcmatch/glob/) form relative to the root of the folder. By convention, most definitions place test assets in a `test-project` folder and this path is referenced in the template `.npmignore` files.
2019-04-11 02:04:21 +03:00
2019-05-18 00:22:14 +03:00
Finally, commit your changes and submit a PR - we'll take a look at it, provide any needed feedback, and then merge it in. We appreciate any and all feedback!
2019-04-11 02:04:21 +03:00
### Release cadence for new containers or container updates
The vscode-dev-containers repo is currently bundled with the VS Code Remote - Containers extension to ensure version compatibility. (There is a feature request to update the container list [out-of-band](https://github.com/microsoft/vscode-remote-release/issues/425), but this is not currently in place.)
Therefore, the release schedule for vscode-dev-containers is the same as the extension. The extension follows the same release cadence and schedule for **VS Code (stable)** as VS Code itself.
* VS Code uses [four week iterations](https://github.com/microsoft/vscode/wiki/Development-Process#inside-an-iteration).
* You can find the dates of the current iteration in a pinned issue in the [vscode repository](https://github.com/microsoft/vscode) and in-flight features for the extension itself in the [vscode-remote-release](https://github.com/microsoft/vscode-remote-release) repository.
The Remote - Containers extension ships into **VS Code Insiders** as new features land during the iteration. This is not currently an automated daily release like VS Code itself, so the exact timing during the iteration can vary. However, it will happen during endgame week at a minimum (week 4).
If there is an urgent need for an update to the definitions outside of this release cycle, please raise an issue in this repository requesting an out-of-band release to fix a critical issue.
2019-04-11 02:04:21 +03:00
## Contributing to Documentation
2019-04-30 02:32:26 +03:00
The majority of VS Code Remote's documentation can be found in the [VS Code docs repository](https://github.com/Microsoft/vscode-docs). This is usually the best place to contribute, but if you have a correction or suggestion for the content found here, we welcome your contributions.
2019-04-11 02:04:21 +03:00
2019-03-28 06:29:57 +03:00
## Reporting Issues
2019-03-27 22:22:15 +03:00
2019-03-28 06:29:57 +03:00
Have you identified a reproducible problem in a dev container definition? We want to hear about it! Here's how you can make reporting your issue as effective as possible.
### Identify Where to Report
2019-04-30 02:32:26 +03:00
This repository is specifically for dev container definitions. If you are not looking to report an issue related to a container definition, you may want to report the issue in the feedback repository for [VS Code Remote Development extensions](https://github.com/Microsoft/vscode-remote-release). Or you may be looking for the [VS Code OSS](https://github.com/Microsoft/vscode) repository. However, note that, the VS Code project is distributed across multiple repositories. See the list of [Related Projects](https://github.com/Microsoft/vscode/wiki/Related-Projects) if you aren't sure which repo is correct.
2019-03-28 06:29:57 +03:00
### Look For an Existing Issue
Before you create a new issue, please do a search in [open issues](https://github.com/Microsoft/vscode-dev-containers/issues) to see if the issue or feature request has already been filed.
Be sure to scan through the [most popular](https://github.com/Microsoft/vscode-dev-containers/issues?q=is%3Aopen+is%3Aissue+label%3Afeature-request+sort%3Areactions-%2B1-desc) feature requests.
If you find your issue already exists, make relevant comments and add your [reaction](https://github.com/blog/2119-add-reactions-to-pull-requests-issues-and-comments). Use a reaction in place of a "+1" comment:
* 👍 - upvote
* 👎 - downvote
If you cannot find an existing issue that describes your bug or feature, create a new issue using the guidelines below.
### Writing Good Bug Reports and Feature Requests
File a single issue per problem and feature request. Do not enumerate multiple bugs or feature requests in the same issue.
Do not add your issue as a comment to an existing issue unless it's for the identical input. Many issues look similar, but have different causes.
The more information you can provide, the more likely someone will be successful at reproducing the issue and finding a fix.
Please include the following with each issue:
* Version of VS Code
* Your operating system
2019-04-03 02:29:43 +03:00
* Include any modifications to container configuration files you've made if possible.
2019-03-28 06:29:57 +03:00
* List of extensions that you have installed
* Reproducible steps (1... 2... 3...) that cause the issue
* What you expected to see, versus what you actually saw
* Images, animations, or a link to a video showing the issue occurring
* A code snippet that demonstrates the issue or a link to a code repository the developers can easily pull down to recreate the issue locally
2019-04-30 02:32:26 +03:00
* **Note:** Because the developers need to copy and paste the code snippet, including a code snippet as a media file (i.e. `.gif`) is not sufficient.
2019-03-27 22:22:15 +03:00
2019-04-30 02:32:26 +03:00
* Errors from the Dev Tools Console (open from the menu: **Help > Toggle Developer Tools**)
2019-03-28 06:29:57 +03:00
### Final Checklist
Please remember to do the following:
* [ ] Search the issue repository to ensure your report is a new issue
* [ ] Recreate the issue after disabling all extensions
* [ ] Simplify your code around the issue to better isolate the problem
Don't feel bad if the developers can't reproduce the issue right away. They will simply ask for more information!
### Follow Your Issue
2019-04-30 02:32:26 +03:00
Once submitted, your report will go into a similar [issue tracking](https://github.com/Microsoft/vscode/wiki/Issue-Tracking) workflow that is used for the VS Code project. Be sure to understand what will happen next, so you know what to expect, and how to continue to assist throughout the process.
2019-03-28 06:29:57 +03:00
2019-03-28 06:32:34 +03:00
### Automated Issue Management
2019-03-28 06:29:57 +03:00
We use a bot to help us manage issues. This bot currently:
* Automatically closes any issue marked `needs-more-info` if there has been no response in the past 7 days.
* Automatically locks issues 45 days after they are closed.
If you believe the bot got something wrong, please open a new issue and let us know.
## Code of Conduct
This project has adopted the [Microsoft Open Source Code of Conduct](https://opensource.microsoft.com/codeofconduct/).
For more information see the [Code of Conduct FAQ](https://opensource.microsoft.com/codeofconduct/faq/) or
contact [opencode@microsoft.com](mailto:opencode@microsoft.com) with any additional questions or comments.
## Thank You!
2019-03-27 22:22:15 +03:00
2019-03-28 06:29:57 +03:00
Your contributions, large or small, make great projects like this possible. Thank you for taking the time to contribute.