-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Config plot option #260
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
Config plot option #260
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
04ac35f
added testing_test_file_tools
aneda 95ee2e4
Added plot_options to config in tools.py and updated session and test
aneda 8a1d1ce
Add a TestCase to stash/restore local creds.
theengineear 961c358
Update test_file_tools to use `PlotlyTestCase`
theengineear 145e290
Specify `file` for credentials and config.
theengineear f93e6df
Stash/restore session in `PlotlyTestCase`.
theengineear 51de393
pep:8ball: oops :)
theengineear 6f2eb8a
Merge pull request #261 from plotly/config-file-safe-tests
59947ec
updated spelling/ names
aneda 93234cb
Merge branch 'config_plot_option' of https://github.com/plotly/python…
aneda 656d4ce
Merge branch 'master' of https://github.com/plotly/python-api into co…
aneda e564a15
fixes this issue
aneda 9cdf568
seems like a typo, changed load to dump
aneda ecb7de8
changed plot_option to world_readable and updated the tests
aneda a1c3d56
pep8
aneda 5328c3f
based on Andrew’s recommendation added decoter to test-grid, so
aneda 2b97d6e
comment edit
aneda 1235039
added the GH issue link
aneda 9c55495
changed kwargs to args for get_config_file
aneda da5faf1
added `@skip` to `test_metadata_to_nonexistent_grid()` so
aneda 2880f4d
update the version
aneda File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Update test_file_tools to use
PlotlyTestCase
- Loading branch information
commit 961c358e58426acb3e69b5a40381a29a395d80f3
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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 sub
2E9C
set 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.
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.
🐄 When you import things from the same place, we usually choose the more compact notation:
from plotly import session, tools