This topic describes how to contribute to the Tableau Server Client (Python) project.

• TOC {:toc}

This section will get you started with the basic workflow, describing how to create your own fork of the repository and how to open a pull request (PR) to add your contributions to the development branch.

1. Make sure you have signed the CLA

2. Fork the repository.

   We follow the "Fork and Pull" model as described here.

3. Clone your fork:

   git clone git@github.com:<user-name>/server-client-python.git

4. Run the tests to make sure everything is peachy:

   python setup.py test

5. Set up the feature, fix, or documentation branch.

   It is recommended to use the format issue#-type-description (e.g. 13-fix-connection-bug) like so:

   git checkout -b 13-feature-new-stuff

Here's a quick checklist to follow when coding to ensure a good pull request (PR):

• Only touch the fewest number of files possible while still accomplishing the goal.
• Ensure all indentation is done as 4-spaces and your editor is set to unix line endings.
• The code matches PEP8 style guides. If you cloned the repo you can run pycodestyle tableauserverclient test samples
• Keep commit messages clean and descriptive. If the PR is accepted it will get 'Squashed' into a single commit before merging, the commit messages will be used to generate the Merge commit message.

All of our tests live under the test/ folder in the repository. We use pytest and the built-in test runner python setup.py test. If a test needs a static file, like a .twb/.twbx, it should live under test/assets/

When adding a new feature or improving existing functionality we may ask that you update the documentation along with your code.

Our documentation is written in markdown and built with Jekyll on GitHub Pages. All of the documentation source files can be found in /docs folder in the gh-pages branch. The docs are hosted on the following URL: https://tableau.github.io/server-client-python

If you are just making documentation updates (adding new docs, fixing typos, improving wording) the easiest method is to use the built-in Edit this file feature right in GitHub (click the pencil icon). If you are viewing one of the pages on https://tableau.github.io/server-client-python, clicking the Edit this page link at the top of the page will take you to the same place.

1. Make a PR as described here against the development branch for code changes, or against gh-pages for documentation updates.

2. Wait for a review and address any feedback. While we try and stay on top of all issues and PRs it might take a few days for someone to respond. Politely pinging the PR after a few days with no response is OK, we'll try and respond with a timeline as soon as we are able.

3. That's it! When the PR has received {:height="5%" width="5%"} (:rocket:'s) from members of the core team they will merge the PR

1. Create an endpoint class for the new feature, following the structure of the other endpoints. Each endpoint usually has get, post, update, and delete operations that require making the url, creating the XML request if necessary, sending the request, and creating the target item object based on the server response.

2. Create an item class for the new feature, following the structure of the other item classes. Each item has properties that correspond to what attributes are sent to/received from the server (refer to docs and Postman for attributes). Some items also require constants for user input that are limited to specific strings. After making all the properties, make the parsing method that takes the server response and creates an instances of the target item. If the corresponding endpoint class has an update function, then parsing is broken into multiple parts (refer to another item like workbook or datasource for example).

3. Add testing by getting real xml responses from the server, and asserting that all properties are parsed and set correctly.

4. Add a sample to show users how to use the new feature.