GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Cleanup samples by graysonarts · Pull Request #74 · tableau/document-api-python

Conversation

@graysonarts

@graysonarts graysonarts commented Aug 29, 2016

Copy link
Copy Markdown
Contributor

In prep for the August release, let's standardize on samples and all lowercase.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Russell Hay added 7 commits August 29, 2016 10:07
Russell Hay
rename Examples to samples
4630d26
Russell Hay
rename GetFields to show-fields
7af3218
Russell Hay
cleanup filenames and casing
80d758d
Russell Hay
clean up list-tds-info
6ee24f5
Russell Hay
cleaning up replicate workbook sample
364aa96
Russell Hay
oops, this should have been underscores
40f9429
Russell Hay
update symlink for world.tds
d540420
@graysonarts graysonarts modified the milestone: 2016.08 release Aug 29, 2016
t8y8
t8y8 reviewed Aug 29, 2016
View reviewed changes
Comment thread .travis.yml
- (cd "Examples/Replicate Workbook" && python replicateWorkbook.py)
- (cd "Examples/List TDS Info" && python listTDSInfo.py)
- (cd "Examples/GetFields" && python show_fields.py)
- (cd "samples/replicate-workbook" && python replicate_workbook.py)

@t8y8 t8y8 Aug 29, 2016

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@t8y8

t8y8 commented Aug 29, 2016

Copy link
Copy Markdown
Contributor

🚀

The names always did drive me a bit crazy, nicely done.

This going in for the .3 release?
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@graysonarts

graysonarts commented Aug 29, 2016

Copy link
Copy Markdown
Contributor Author

Yup, it's going into 0.3, oh I should scrub the readme to make sure I don't need to update anything there.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@graysonarts
graysonarts merged commit 9126049 into tableau:development Aug 29, 2016
@graysonarts
graysonarts deleted the cleanup-samples branch August 29, 2016 20:14
graysonarts pushed a commit that referenced this pull request Aug 31, 2016
Russell Hay
Prepping for 0.3 release (#75)
ff372f4 
* Prepping for 0.3 release

* Cleanup samples (#74)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

2016.08 release (v0.3)

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@graysonarts @t8y8
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.