Skip to content

Latest commit

 

History

History
125 lines (89 loc) · 6.38 KB

CONTRIBUTING.md

File metadata and controls

125 lines (89 loc) · 6.38 KB

Contributing

Thanks for contributing! Our community of developers is what put SmartThings on the map!

Development

npm Workspaces

This is the monorepo for the SmartThings CLI. Currently, the following packages are included:

  • cli - the CLI itself; @smartthings/cli node package
  • lib - a library for use in the CLI and its extensions; @smartthings/cli-lib node package
  • testlib - a library for use in the CLI and its extensions with utility methods to make testing with Jest easier; @smartthings/cli-testlib node package

Build

  1. Be sure you're using at least Node.js version 16.
  2. run npm install
  3. run npm run compile
  4. To run the CLI that was just compiled, run the run command in packages/cli/bin. You can create a link to this file to make it easier to run. Since the final installed name will be "smartthings", that's a good name for the link. For example: ln -s ~/mydevdir/smartthings-cli/packages/cli/bin/run ~/bin/smartthings

Other useful scripts:

  • run npm run watch to watch for changes and compile on the fly
  • run npm run build to clean and compile
  • run npm run format to automatically fix up lint issues if possible
  • run npm run full-clean to start over again. This is sometimes helpful when pulling new code.

Before opening a pull request be sure to:

  1. Run eslint via npm run lint
  2. Run tests with npm run test
  3. If you've added or or removed commands or updated any of their arguments or flags, update the readme with npm run readme.

Changesets

We use changesets to automatically version and publish releases, and generate release notes. Please include one with your pull request by following the instructions to add a changeset to a multi-package repository. We have a bot that will remind you to include one with your pull request if you forget.

Core SDK

The CLI depends on the SmartThings Core SDK. To use a pre-release version of the SDK for testing purposes, you'll need to make any required changes in both checked out repositories and then npm-link the SDK into the CLI.

How Can I Contribute?

Improve Documentation

As a user of the SmartThings CLI, you're the perfect candidate to help us improve our documentation. Error fixes, typo corrections, better explanations, more examples, etc. Open issues for things that could be improved – anything. Even improvements to this document.

Give Feedback on Issues

We're always looking for more opinions on discussions in the issue tracker. It's a good opportunity to influence the future direction of the CLI.

Submitting an Issue or Feature Request

  • Search the issue tracker before opening an issue
  • Ensure you're using the latest version
  • Use a clear and descriptive title
  • Include as much information as possible by filling out the issue template
  • The more time you put into an issue, the more we will

Submitting a Pull Request

  • Non-trivial changes are often best discussed in an issue first, to prevent you from doing unnecessary work.
  • For ambitious tasks, you should try to get your work in front of the community for feedback as soon as possible. Open a pull request as soon as you have done the minimum needed to demonstrate your idea. At this early stage, don't worry about making things perfect, or 100% complete. Describe what you still need to do and submit a draft pull request. This lets reviewers know not to nit-pick small details or point out improvements you already know you need to make.
  • Don't include unrelated changes
  • New features should be accompanied with tests and documentation
  • Commit messages
    • Use a clear and descriptive title for the pull request and commits
    • Commit messages must be formatted properly using conventional commit format. Our CI will check this and fail any PRs that are formatted incorrectly.
    • This repo is commitizen friendly, so you can use the cz cli to help create your commits.
  • Lint and test before submitting the pull request
    • npm run lint
    • npm run test
  • Write a convincing description of why we should land your pull request. Answer why it's needed and provide use-cases.
  • Make the pull request from a topic branch (not master)
  • You might be asked to do changes to your pull request. There's never a need to open another pull request – just update the existing one.
  • Don't forget to generate a changeset

Finding Contributions to Work On

Look at the existing issues for areas of contribution. Searching for issues labeled help wanted would be a great place to start.

More About SmartThings

If you are not familiar with SmartThings, we have extensive online documentation.

To create and manage your services and devices on SmartThings, create an account in the developer workspace.

The SmartThings Community is a good place share and ask questions.

There is also a SmartThings reddit community where you can read and share information.

License and Copyright

Licensed under the Apache License, Version 2.0

Copyright 2022 SmartThings, Inc.