Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ZERO circular dependency #2429

Merged
merged 17 commits into from
Mar 2, 2018
Merged

ZERO circular dependency #2429

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
bee37dd
move grid module to components/
etpinard Mar 1, 2018
973abbc
merge src/plot_api/register.js into src/registry.js
etpinard Mar 1, 2018
b08c402
fixes #621 - make x/y/z dependent error bar attributes
etpinard Mar 1, 2018
87275af
use errorbars methods via getComponentMethod
etpinard Mar 1, 2018
52b83b3
add apiMethodRegistry in src/registry
etpinard Mar 1, 2018
9fc8832
add Registry.call method + use it to :hocho: circular deps !!!
etpinard Mar 1, 2018
381b6e4
move Plotly.setPlotConfig to plot_api.js
etpinard Mar 1, 2018
ed8d56d
use Registry.traceIs instead of plots.traceIs copy
etpinard Mar 1, 2018
204799c
:hocho: src/plotly.js !!!
etpinard Mar 1, 2018
2c529ec
:lock: down circular dep test at :zero:
etpinard Mar 1, 2018
c6df699
update tests (mostly update ref to spyOn modules)
etpinard Mar 1, 2018
011443e
add `flaky` tag to scattergl visible toggle test
etpinard Mar 1, 2018
9246fda
grab Grid module method using Registry.getComponentMethod
etpinard Mar 2, 2018
d27b3ba
fix typo in test_syntax
etpinard Mar 2, 2018
e05f4f0
rename register_test -> registry_test & improve it!
etpinard Mar 2, 2018
705aed6
replace Registry.call('', []) with Registry.call('', ...[])
etpinard Mar 2, 2018
a851aa5
no need for Registry in plot_api/to_image
etpinard Mar 2, 2018
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
7 changes: 2 additions & 5 deletions tasks/test_syntax.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,12 +260,9 @@ function assertCircularDeps() {
var circularDeps = res.circular();
var logs = [];

// see https://github.com/plotly/plotly.js/milestone/9
var MAX_ALLOWED_CIRCULAR_DEPS = 16;

if(circularDeps.length > MAX_ALLOWED_CIRCULAR_DEPS) {
if(circularDeps.length) {
console.log(circularDeps.join('\n'));
logs.push('some new circular dependencies were added to src/');
logs.push('some circular dependencies were found to src/');
Copy link
Collaborator

Choose a reason for hiding this comment

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

found in src/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good 👁️ done in d27b3ba

}

log('circular dependencies: ' + circularDeps.length, logs);
Expand Down