Skip to content

Commit

Permalink
Merge branch 'main' of https://github.com/kedro-org/kedro-viz into fe…
Browse files Browse the repository at this point in the history
…ature/kedro-viz-lite
  • Loading branch information
ravi-kumar-pilla committed Jul 22, 2024
2 parents e120ccc + faceed9 commit a711cf0
Show file tree
Hide file tree
Showing 32 changed files with 593 additions and 471 deletions.
2 changes: 1 addition & 1 deletion .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ updates:
directory: "/package" # Location of package manifests
schedule:
interval: "monthly"
versioning-strategy: increase-if-necessary
versioning-strategy: auto
ignore:
- dependency-name: "types-*"
labels:
Expand Down
11 changes: 6 additions & 5 deletions .gitpod.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,17 @@ tasks:
- name: backend
before: echo PIP_USER=no >> ~/.bashrc && export PIP_USER=no
init: |
pip install "pip==23.3.1" --user
pip install -r package/test_requirements.txt -r demo-project/src/docker_requirements.txt --user
pip install uv
uv pip install "pip==23.3.1" --system
uv pip install -r package/test_requirements.txt -r demo-project/src/docker_requirements.txt --system
# Install latest kedro and all its dependencies.
pip install https://github.com/kedro-org/kedro/archive/main.zip --user
uv pip install https://github.com/kedro-org/kedro/archive/main.zip --system
# force-reinstall kedro to ensure that the latest version from main is installed even when
# its version number is the same as that specified in docker_requirements.txt.
pip install https://github.com/kedro-org/kedro/archive/main.zip --user --force-reinstall --no-deps
uv pip install https://github.com/kedro-org/kedro/archive/main.zip --force-reinstall --no-deps --system
command: |
gp sync-await build_complete
pip install -e package --no-deps
uv pip install -e package --no-deps --system
gp sync-done kedro_installed
make run
Expand Down
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ Working on your first pull request? You can learn how from these resources:

- Fork the project
- Develop your contribution in a new branch and open a PR against the `main` branch
- Make sure the CI builds are green (have a look at the section [Running checks locally](#running-checks-locally) below)
- Make sure the CI builds are green (have a look at the section [Running checks locally](#testing-guidelines) below)
- Update the PR according to the reviewer's comments

# Contribution guidelines
Expand Down
4 changes: 2 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ package:
find . -regex ".*/__pycache__" -exec rm -rf {} +
find . -regex ".*\.egg-info" -exec rm -rf {} +
test -f package/kedro_viz/html/index.html || (echo "Built npm package not found; packaging process cancelled."; exit 1)
cd package && python setup.py clean --all
cd package && python setup.py sdist bdist_wheel
cd package && rm -rf build/ dist/
cd package && python -m build

build:
rm -rf build package/build package/dist package/kedro_viz/html pip-wheel-metadata package/kedro_viz.egg-info
Expand Down
293 changes: 0 additions & 293 deletions README.md

This file was deleted.

1 change: 1 addition & 0 deletions README.md
4 changes: 4 additions & 0 deletions RELEASE.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,11 @@ Please follow the established format:

- Relax `packaging` pin in requirements. (#1947)
- Add favicon to kedro-viz documentation. (#1959)
- Add "-p" flag to kedro-viz to match kedro run. (#1960)
- Fix bug related to nested namespace pipelines. (#1897)
- Migrate from `toposort` to `graphlib`. (#1942)
- Fix packaging. (#1766)
- Adjust requirements file and dependabot versioning strategy. (#1978)

# Release 9.1.0

Expand Down
27 changes: 27 additions & 0 deletions cypress/support/commands.js
Original file line number Diff line number Diff line change
Expand Up @@ -235,3 +235,30 @@ Cypress.Commands.add(
.click();
}
);

/**
* Custom command to wait for page load before enabling pretty names
*/
Cypress.Commands.add('__waitForSettingsButton__', () => {
cy.get('[data-test="Change the settings flags"]', { timeout: 20000 }).should('be.visible');
});

/**
* Custom command to enable pretty name
*/
Cypress.Commands.add('enablePrettyNames', () => {

// Wait for the settings button to be visible
cy.__waitForSettingsButton__();

// Visit the settings panel
cy.get('[data-test="Change the settings flags"]').click();

// Enable the pretty names setting
cy.get('[data-test="pipeline-toggle-input-isPrettyName"]').check({ force: true });

// Apply changes and close the settings panel
cy.get('[data-test="Apply changes and close in Settings Modal"]').click({
force: true,
});
});
Loading

0 comments on commit a711cf0

Please sign in to comment.