Contributing to TDS - developer guide
Set up your environment
To get started, clone the repository and create your branch from master. If you are not part of the TELUS digital organization, you may fork the repository instead.
After setting up TDS locally, the following steps will get you started:
# Bootstrap your dev environment yarn bootstrap
We use Prettier, an opinionated code formatter that integrates with your text editor. Configure your IDE or text editor to format your code automatically on save, and Prettier will adjust your syntax in accordance with the TDS conventions.
Set up your dev environment
# Start the styleguidist dev server, check output for the location of the docs yarn dev # Open a new terminal window # Start the test watcher, it will rerun tests based on the files you modify yarn test:watch
After this, you can open up a browser to view the documentation site (usually http://localhost:6060). The browser will automatically refresh when there are changes to any of the source files.
Write some code
Read the codebase overview to understand the structure of the codebase and the conventions being followed.
If you need to create a new component, run the scaffolding script to generate the basic folder structure, React component, documentation, and base unit tests for a new component.
yarn scaffold MyComponent
Running and updating tests
All TDS components use a combination of Jest tests and Nightwatch visual regression tests. As part of our Git hooks, these tests are run automatically on commit and on push. However, there are cases where you may want to run these tests manually, or require the ability to update test snapshots that are no longer up to date with the component you're working on.
Jest (unit tests)
Jest unit tests are integrated into all TDS React components. These are run to ensure that a component's functionality has not been compromised by a change. These unit tests will check the component's different states by providing different sets of prop values, and compare them to a set of pre-defined criteria. It is important to create new unit tests whenever a feature is added or significantly modified to ensure the stability of the component.
If a component's structure has significantly changed, it may require a snapshot update with the update command listed below. However, it is important to use discretion here, as a failing snapshot may be the symptom of a larger problem with the component. (Such as unexpected extra/missing classes or unintended shuffling of the component's DOM structure) Always review the log of a snapshot failure before running the update command. When in doubt, reach out to our support avenues.
# Manually run unit tests on modified components yarn test # Options: # -a: Run tests on all components regardless of if they were recently modified. # -u: Update test snapshots. (Useful if the component's structure has changed)
Nightwatch e2e tests are run to ensure that no unexpected visual regressions were made to a component. These tests are run automatically on all components with no test writing required on the developer's part.
Before tests can be run, the TDS docs must be running locally on your machine in e2e mode.
Once the local server is up and running, you may now run e2e tests. These tests will fail if the component's appearance has diverged from its saved screenshot. Visual modifications to a component will almost always require screenshots to be updated.
# Manually run visual regression tests on modified components yarn test:e2e # Options: # -a: Run tests on all components regardless of if they were recently modified. # -u: Update test screenshots. (Useful if the component was purposefully changed visually)
Make a commit
husky is used to run precommit tasks on staged files, which includes code formatting, linting, and tests. You will not be able to make a commit until the precommit tasks pass. We also have a prepush hook to run a full build before pushing your code.
Automated component versioning is facilitated by the Conventional Commits specification, so you must be deliberate when choosing the type of commit.
Given the commit format:
type(scope): subject body footer
type field to inform lerna what Conventional Commit you intend to use:
- For breaking changes, use the feat commit type with body text that begins with the phrase "BREAKING CHANGE:"
- For minor changes, use the feat commit type
- For patches, use the fix commit type
Follow these guidelines to determine the type of your commit:
- Breaking changes are removals of features such as props, changes that affect the box model, or dramatic changes in branding or appearance
- Minor changes are new features, animations, props, or visual options
- Patches are defect fixes that do not remove features, alter pixel dimensions related to the box model, nor add new features. If an intended feature was not working in a previous release, changing that feature to match the original design counts as a patch even if it affects the box model
scope field when referring to an area in the codebase, such as a package name (e.g.
a directory (e.g.
subject field to write a succinct description of the change. The first word is often a verb using present
imperative tense such as 'change' or 'remove'.
body field can include more detailed notes regarding the change. If there is a breaking change, it must begin with
the phrase 'BREAKING CHANGE:'.
footer field can be used to reference a commit hash or issue number on GitHub.
|feat||Indicates a minor, public-facing change. A change is considered minor if it adds new functionality in a backwards-compatible manner. It must also be perceivable by consumers of TDS Core, and not just a change that only impacts developers of TDS Core. Additionally, it may be used if functionality is being deprecated. If a commit of this type contains a public-facing breaking change, add the text
|fix||Indicates a patch level, public-facing change. Generally, this is used for backwards-compatible bug fixes. There should be no consumer-facing API changes at all in patches. It must also be perceivable by consumers of TDS, and not just a change that only impacts developers of TDS Core.|
|test||Used for commits that only modify/add unit tests or anything test related, such as snapshots, screenshots, or test scripts.|
|refactor||Used if code has been refactored without any modifications to functionality or behaviour. If your refactor also fixes a bug, please use the
|docs||Indicates a modification to documentation. Changes to the .md files of components, or changes to the documentation site use this type.|
|chore||A type for miscellaneous changes. Anything not covered here is considered a chore. Additionally, this is useful when iterating on changes due to PR reviews or other factors. Only one commit may be labeled as a
The TDS codebase includes a script to run commitizen for a streamline commit-making experience:
# Stage your files and make a commit using commitizen yarn cz ## At the `type` and `scope` prompts, you can press TAB to view options
Example commit messages
feat(core-flex-grid): remove center prop BREAKING CHANGE: deprecated `center` prop is removed
fix(core-button-link): adjust hover animation speed
Make a Pull Request
Before making your Pull Request, we have a pre-pr tasks as a final verification step. You may make your PR only after its successful completion.
Paste the full output of the pre-pr task into the body of your PR so that a maintainer/publisher can verify when publishing.
The pre-pr task will show you the version change that will result from your changeset. If the output is unexpected, you may need to adjust your commit messages before making your PR. See the Conventional Commits spec FAQ for more info on correcting mistakes.