# Contributing to Swift-DocC ## Introduction ### Welcome Thank you for considering contributing to Swift-DocC. Please know that everyone is welcome to contribute to Swift-DocC. Contributing doesn’t just mean submitting pull requests—there are many different ways for you to get involved, including answering questions on the [Swift Forums](https://forums.swift.org/c/development/swift-docc), reporting or screening bugs, and writing documentation. No matter how you want to get involved, we ask that you first learn what’s expected of anyone who participates in the project by reading the [Swift Community Guidelines](https://swift.org/community/) as well as our [Code of Conduct](/CODE_OF_CONDUCT.md). This document focuses on how to contribute code and documentation to this repository. ### Legal By submitting a pull request, you represent that you have the right to license your contribution to Apple and the community, and agree by submitting the patch that your contributions are licensed under the Apache 2.0 license (see [`LICENSE.txt`](/LICENSE.txt)). ## Contributions Overview Swift-DocC is an open source project and we encourage contributions from the community. ### Contributing Code and Documentation Before contributing code or documentation to Swift-DocC, we encourage you to first open a [GitHub issue](https://github.com/apple/swift-docc/issues/new/choose) for a bug report or feature request. This will allow us to provide feedback on the proposed change. However, this is not a requirement. If your contribution is small in scope, feel free to open a PR without first creating an issue. All changes to Swift-DocC source must go through the PR review process before being merged into the `main` branch. See the [Code Contribution Guidelines](#code-contribution-guidelines) below for more details. ## Building Swift-DocC ### Prerequisites Swift-DocC is a Swift package. If you're new to Swift package manager, the [documentation here](https://swift.org/getting-started#using-the-package-manager) provides an explanation of how to get started and the software you'll need installed. ### Build Steps 1. Checkout this repository using: ```bash git clone git@github.com:apple/swift-docc.git ``` 2. Navigate to the root of your cloned repository with: ```bash cd swift-docc ``` 3. Create a new branch off of `main` for your change using: ```bash git checkout -b branch-name-here ``` Note that `main` (the repository's default branch) will always hold the most recent approved changes. In most cases, you should branch off of `main` when starting your work and open a PR against `main` when you're ready to merge that work. 4. Build Swift-DocC from the command line by running: ```bash swift build ``` Alternatively, to use Xcode, open the `Package.swift` file at the repository's root. Then, build it by pressing Command-B. ### Run Steps You can run your newly built version of `docc` with: ```bash swift run docc ``` Or, in Xcode, run the `docc` [scheme](https://developer.apple.com/library/archive/documentation/ToolsLanguages/Conceptual/Xcode_Overview/ManagingSchemes.html). ### Miscellaneous The JSON output of Swift-DocC is optimized for file size. If you need to inspect it for debugging purposes set the `DOCC_JSON_PRETTYPRINT` environment variable to "YES" to enable pretty printing. ```bash export DOCC_JSON_PRETTYPRINT="YES" ``` ## Code Contribution Guidelines ### Overview - Do your best to keep the git history easy to understand. - Use informative commit titles and descriptions. - Include a brief summary of changes as the first line. - Describe everything that was added, removed, or changed, and why. - All changes must go through the pull request review process. - Follow the [Swift API Design guidelines](https://swift.org/documentation/api-design-guidelines/). ### Pull Request Preparedness Checklist When you're ready to have your change reviewed, please make sure you've completed the following requirements: - [x] Add tests to cover any new functionality or to prevent regressions of a bug fix. - [x] Run the `/bin/test` script and confirm that the test suite passes. (See [Testing Swift-DocC](#testing-swift-docc).) - [x] Add source code documentation to all added or modified APIs that explains the new behavior. ### Opening a Pull Request When opening a pull request, please make sure to fill out the pull request template and complete all tasks mentioned there. Your PR should mention the number of the GitHub issue your work is addressing. Most PRs should be against the `main` branch. If your change is intended for a specific release, you should also create a separate branch that cherry-picks your commit onto the associated release branch. ### Code Review Process All PRs will need approval from someone on the core team (someone with write access to the repository) before being merged. All PRs must pass the required continuous integration tests as well. If you have commit access, you can run the required tests by commenting the following on your PR: ``` @swift-ci Please smoke test ``` If you do not have commit access, please ask one of the code owners to trigger them for you. For more details on Swift-DocC's continuous integration, see the [Continous Integration](#continuous-integration) section below. ## Testing Swift-DocC Swift-DocC is committed to maintaining a high level of code quality. Before opening a pull request, we ask that you: 1. Run the full test suite and confirm that it passes. 2. Write new tests to cover any changes you made. The test suite can be run with the provided [`test`](/bin/test) script by navigating to the root of the repository and running the following: ```bash bin/test ``` By running tests locally with the `test` script you will be best prepared for automated testing in CI as well. ### Testing in Xcode You can test a locally built version of Swift-DocC in Xcode 13 or later by setting the `DOCC_EXEC` build setting to the path of your local `docc`: 1. Select the project in the Project Navigator. 2. In the Build Settings tab, click '+' and then 'Add User-Defined Setting'. 3. Create a build setting `DOCC_EXEC` with the value set to `/path/to/docc`. The next time you invoke a documentation build with the "Build Documentation" button in Xcode's Product menu, your custom `docc` will be used for the build. You can confirm that your custom `docc` is being used by opening the latest build log in Xcode's report navigator and expanding the "Compile documentation" step. ### Using Docker to Test Swift-DocC for Linux Today, Swift-DocC supports both macOS and Linux. While most Swift APIs are cross-platform, there are some minor differences. Because of this, all PRs will be automatically tested in both macOS and Linux environments. macOS users can test that their changes are compatible with Linux by running the test suite in a Docker environment that simulates Swift on Linux. 1. Install [Docker Desktop for Mac](https://www.docker.com/products/docker-desktop). 2. Build Swift-DocC (see [Building Swift-DocC](#building-swift-docc)). 3. Run the following command from the root of this repository to build the Swift-DocC Docker image: ```bash docker build -t swift-docc:latest . ``` 4. Run the following command to run the test suite: ```bash docker run -v `pwd`:/swift-docc swift-docc sh -c 'swift test --package-path /swift-docc --parallel --skip-update' ``` 5. To interactively test the command line interface, first log into the container with: ```bash docker run -i -t -v `pwd`:/swift-docc swift-docc /bin/bash ``` And then run `docc` within the container: ```bash cd swift-docc swift run docc ``` ## Continuous Integration Swift-DocC uses [swift-ci](https://ci.swift.org) infrastructure for its continuous integration testing. The tests can be triggered on pull-requests if you have commit access. If you do not have commit access, please ask one of the code owners to trigger them for you. 1. **Smoke Test _(required)_:** Run the project's unit tests on macOS and Linux by commenting the following: ``` @swift-ci Please smoke test ``` This is **required** before a pull-request can be merged.
Platform specific instructions: 1. Run the project's unit tests on **macOS** by commenting the following: ``` @swift-ci Please smoke test macOS platform ``` 2. Run the project's unit tests on **Linux** by commenting the following: ``` @swift-ci Please smoke test Linux platform ```
2. **Test:** Run the project's unit tests on macOS and Linux, along with a selection of compatibility suite tests on macOS by commenting the following: ``` @swift-ci Please test ```
Platform specific instructions: 1. Run the project's unit tests on **macOS**, along with a selection of compatibility suite tests by commenting the following: ``` @swift-ci Please test macOS platform ``` 2. Run the project's unit tests on **Linux** by commenting the following: ``` @swift-ci Please test Linux platform ``` > **Note**: This is equivalent to running smoke tests for the Linux platform.
3. **Compatibility Test:** Run the Swift compatibility suite tests in both Release and Debug configuration by commenting the following: ``` @swift-ci Please test source compatibility ```
Build configuration specific instructions: 1. Run the Swift compatibility suite tests in **Release** configuration by commenting the following: ``` @swift-ci Please test source compatibility Release ``` 2. Run the Swift compatibility suite tests in **Debug** configuration by commenting the following: ``` @swift-ci Please test source compatibility Debug ```
4. **Performance Smoke Test:** Test compiler performance on macOS with **a selection** of Swift compatibility suite projects by commenting the following: ``` @swift-ci Please smoke test compiler performance ``` 5. **Performance Test:** Test compiler performance on macOS with **all** Swift compatibility suite projects by commenting the following: ``` @swift-ci Please test compiler performance ``` ## Your First Contribution Unsure of where to begin contributing to Swift-DocC? You can start by looking at the issues on the [good first issue](https://github.com/apple/swift-docc/contribute) page. Once you've found an issue to work on, follow the above instructions for [Building Swift-DocC](#building-swift-docc).