Fix CircleCI build failure due to secret scan and missing toposort dependency

ravi-kumar-pilla commented 6 months ago

Description

Resolves the build failure - https://app.circleci.com/pipelines/github/kedro-org/kedro-viz/11807/workflows/c0708990-c866-4e6f-b59d-b4bb94068c36/jobs/75254

Development notes

Added src/components/feature-hints/feature-hints-content.js file to the trufflehog ignore text
Added temporary dependency for toposort as Kedro has dropped the dependency - https://github.com/kedro-org/kedro/pull/3728

QA notes

CircleCI build should pass
All tests should pass

Checklist

[x] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes

astrojuanlu commented 6 months ago

I see there's an actual dependency on toposort

https://github.com/kedro-org/kedro-viz/blob/6206e436029f457386adea67bc2af99b194ae557/package/kedro_viz/services/layers.py#L6

ravi-kumar-pilla commented 6 months ago

I see there's an actual dependency on toposort

https://github.com/kedro-org/kedro-viz/blob/6206e436029f457386adea67bc2af99b194ae557/package/kedro_viz/services/layers.py#L6

yes @astrojuanlu . I think we were using this dependency from Kedro environment. Now that Kedro has dropped it, I added the requirement to our requirements file and will create a ticket to see if we can migrate to graphlib completely. Thanks

astrojuanlu commented 6 months ago

Don't block this one on my review, I'll be in meetings all day 😅

kedro-org / kedro-viz