Skip to content

Update the dev guide and minor docs fixes #105

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 31, 2016
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/_config.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Site settings
title: Tableau Document API
email: jdominguez@tableau.com
email: github@tableau.com
description: Programmatically update your Tableau workbooks and data sources.
baseurl: "/document-api-python"
permalinks: pretty
Expand Down
2 changes: 1 addition & 1 deletion docs/_layouts/docs.html
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
<div class="content .col-xs-12 .col-sm-8 .col-md-9">
<h1>{{ page.title }}</h1>
<div class="edit-container">
<a href="https://github.com/tableau/document-api-python/edit/master/{{ page.path }}" class="edit-links"><span class="glyphicon glyphicon-pencil"></span> Edit this page</a>
<a href="https://github.com/tableau/document-api-python/edit/master/docs/{{ page.path }}" class="edit-links"><span class="glyphicon glyphicon-pencil"></span> Edit this page</a>
&nbsp;
<a href="https://github.com/tableau/document-api-python/issues" class="edit-links"><span class="glyphicon glyphicon-flag"></span> Submit an issue</a>
</div>
Expand Down
51 changes: 33 additions & 18 deletions docs/docs/dev-guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,51 +3,66 @@ title: Developer Guide
layout: docs
---

### Making your first patch
## Submitting your first patch

1. Make sure you've signed the CLA
1. Make sure you have [signed the CLA](http://tableau.github.io/#contributor-license-agreement-cla)

1. Clone the repo
1. Fork the repository.

We follow the "Fork and Pull" model as described [here](https://help.github.com/articles/about-collaborative-development-models/).

1. Clone your fork:

```shell
git clone http://github.com/tableau/document-api-python
git clone http://github.com/<your_username>/document-api-python
```

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

```shell
python setup.py test
```

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

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

```shell
git checkout -b 13-feature-new-stuff
```

1. Code and Commit!
1. Code and commit!

Here's a quick checklist for ensuring a good pull request:

- Only touch the minimal amount 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 .`
- 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.

Here's a quick checklist for ensuring a good diff:
1. Add tests.

- The diff touches the minimal amount of files possible while still fufilling the purpose of the diff
- The diff uses Unix line endings
- The diff adheres to our PEP8 style guides. If you've cloned the repo you can run `pycodestyle .`
All of our tests live under the `test/` folder in the repository.
We use `unittest` 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/`

1. Add Tests
1. Update the documentation.

1. Update Documentation
Our documentation is written in markdown and built with Jekyll on Github Pages. All of the documentation source files can be found in `docs/docs`.

Our documentation is written in markdown and built with [Mkdocs](http://www.mkdocs.org). More information on how to update and build the docs can be found [here](#updating-documentation)
When adding a new feature or improving existing functionality we may ask that you update the documentation along with your code.

If you are just making a PR for documentation updates (adding new docs, fixing typos, improving wording) the easiest method is to use the built in `Edit this file` in the Github UI

1. Run the tests again and make sure they pass!
1. Submit to your fork.

1. Submit to your fork
1. Make a PR as described [here](https://help.github.com/articles/creating-a-pull-request-from-a-fork/) against the 'development' branch.

1. Submit a PR
1. 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.

1. Wait for a review, and address any feedback.
1. That's it! When the PR has received :rocket:'s from members of the core team they will merge the PR

<!--
### Updating Documentation
Expand Down
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy