Skip to content

CONTRIBUTING.md

Latest commit

 

History

History
63 lines (54 loc) · 6.98 KB

CONTRIBUTING.md

File metadata and controls

63 lines (54 loc) · 6.98 KB
Raw

Contributing to Design System React

First, on behalf of the core maintainers, I'd like to thank you for wanting to contribute and make the user experience for your end-users better and improve the developer experience of this library. 👍🎉 -- @interactivellama

Contributing process

  1. Read the Codebase Overview to learn concepts and best practices for the codebase and to confirm contribution is within project scope.
  2. Create a new issue before starting your solution to keep track of what you want to contribute, so that others can offer suggestions, collaborate on a public API (props), or let you know if there is already an effort in progress.
  3. Fork this repository, clone your fork locally. Create a topic branch locally.
  4. Add a new component folder to the /components.
  5. Add all public components exports to /components/index.js and the component's name to /components/constants.js
  6. Add documentation site examples and dev storybook stories in /components/[COMPONENT]/__examples__/.
  7. Hook up storybook and site examples in /components/[COMPONENT]/__docs__/. Import these examples into Storybook within /components/storybook-stories.js and /components/site-stories.js respectively. Run npm start to view stories at localhost:9001. Site examples only have access to variables exported in /components/index.js, so you should limit your component's site example imports to these variables. See #1192 for more information.
  8. Copy new examples from /components/storybook-stories.js to /components/story-based-tests.js. This will add DOM and image snapshot testing for the component. These tests use Storyshots and are run without a DOM. Most props that don't involve the user can be tested here.
  9. Add callback prop tests in Mocha test framework to /components/[COMPONENT]/__tests__/
  10. Push to your username's forked repository.
  11. Send us a well-documented pull request targeting master from your forked repository. GitHub pull requests should have a descriptive title, a brief summary, @mention several relevant people to review the code, add helpful GitHub comments on lines where you have questions or concerns. All contributors must sign a Contributor License Agreement.
  12. We'll review your code, suggest any needed changes, and hopefully merge it in soon. Thank you!

Contributing Guidelines

  • Are you a first-time contributor? If you would like a simple task to start out with, you might consider these issues or run npm run lint and fix a few warnings.
  • UX pattern / design must exist in SLDS. Components in the process of being added to SLDS will be considered as prototypes.
  • All new props and components need tests. Please review the testing readme
  • Contributions of components with a subset of SLDS variants will be considered. Please consider your architecture in view of the other variants and create an issue before starting just to be certain.
  • Follow this library's prettier-eslint settings. npm run lint:fix will run Prettier and then ESlint and write changes to your files.
    • You can enable this behavior at save in your editor, too. For instance, in Visual Studio Code, run the prettier extension and set "editor.formatOnSave": true and prettier.eslintIntegration: true.
  • If you are adding a feature, add a story to the React Storybook that uses your feature, so that reviewers can test it.
  • Add enough Storybook stories and testing examples to show use of all component prop and values--if they are enumerated. All examples that are present for a component in the SLDS website should be created as a Storybook story and imported into the documentaiton site examples.
  • Prop description tables on the documentation site are generated from propType comments within the component. Use npm run build-docs to confirm comment compatibility. Introductory component descriptions are generated from the comment directly before the component declaration with react-docgen.
  • All props descriptions should have a Tested with snapshot testing. or Tested with Mocha framework. notice in them.

The review process

  1. Read through the modified/added code in the pull request.
  2. git clone this repository. Pull down the pull requested branch. It will be within the contributor's forked repository. For instance, git checkout -b interactivellama-data-table-width master then git pull git@github.com:interactivellama/design-system-react.git data-table-width. You could also create an additional remote and pull down the branch directly.
  3. Run npm install and npm start.
  4. Review the appropriate Storybook stories at http://localhost:9001/.
  5. Review tests. Open http://localhost:8001/ and confirm that tests are passing in your browser.
  6. Compare component markup to SLDS markup. Reviewing the snapshot strings is the easiest way. Add year-first date and commit SHA to last-slds-markup-review in package.json and push to pull request branch.
  7. Request a review of the new component/feature by the Salesforce UX Accessibility Team. Add year-first review date, and commit SHA, last-accessibility-review, to package.json and push to pull request branch. 
    {
  "component": "",
  "status": "prod",
  "display-name": "",
  "last-accessibility-review": {
    "date-iso-8601": "2017/12/31",
    "commit-sha": ""
  },
  "last-slds-markup-review": {
    "date-iso-8601": "2017/12/30",
    "commit-sha": ""
  },
  "SLDS-component-path": ""
},
  8. While the contributor's branch is checked out, run npm run local-update within locally cloned site repo to confirm the site will function correctly at the next release. This will also build the bundle (npm run dist) and use the bundle in the documentation site and confirm that the bundle works.

Testing the documentation site (internal)

  1. Pull down the documentation site (currenly private) and place in the same parent folder as this library: git clone git@github.com:salesforce-ux/design-system-react-site.git and run npm install.
  2. Run npm run local-update from within design-system-react-site to build, copy, and serve a local version of this library into the site. You should be able to now view the updated site at http://localhost:8080/ and resolve any issues with updated documentation.