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

Release v6.5.0 #1513

Merged
merged 6 commits into from
Aug 31, 2023
Merged

Release v6.5.0 #1513

merged 6 commits into from
Aug 31, 2023

Conversation

rashidakanchwala
Copy link
Contributor

@rashidakanchwala rashidakanchwala commented Aug 31, 2023 •
edited
Loading

Description

This is a minor release since we are adding support for Python 3.11 and removing upper-bounds on Python.

Major features and improvements

Bug fixes and other changes

Development notes

QA notes

View the Gitpod link and make sure all is good. Please also check the update reminders section.

Checklist

  • 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

@rashidakanchwala rashidakanchwala marked this pull request as draft August 31, 2023 12:32
@rashidakanchwala rashidakanchwala marked this pull request as ready for review August 31, 2023 12:47
merelcht
merelcht approved these changes Aug 31, 2023
View reviewed changes
Copy link
Member

@merelcht merelcht left a comment

Choose a reason for hiding this comment

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

image

ravi-kumar-pilla
ravi-kumar-pilla approved these changes Aug 31, 2023
View reviewed changes
Copy link
Contributor

@ravi-kumar-pilla ravi-kumar-pilla left a comment

Choose a reason for hiding this comment

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

Awesome :)

@ravi-kumar-pilla ravi-kumar-pilla merged commit f5ffa7a into main Aug 31, 2023
@ravi-kumar-pilla ravi-kumar-pilla deleted the release/6.5.0 branch August 31, 2023 16:03
ravi-kumar-pilla pushed a commit that referenced this pull request Aug 31, 2023
@rashidakanchwala @ravi-kumar-pilla
Release v6.5.0 (#1513)
d527151 
* v6.5.0

* release

* update-reminder-content

* update reminder

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>
vladimir-mck pushed a commit that referenced this pull request Sep 6, 2023
@rashidakanchwala @vladimir-mck
Release v6.5.0 (#1513)
8791522 
* v6.5.0

* release

* update-reminder-content

* update reminder

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>
ravi-kumar-pilla added a commit that referenced this pull request Sep 11, 2023
@ravi-kumar-pilla @rashidakanchwala
@noklam @jitu5
Fix for Kedro Viz Connection Error (#1507)
335be7d 
* initial draft for resolving connection error

* refactor launchers and test code

* modify unit tests

* fix lint errors

* fix run_viz tests

* update unit test for coverage

* update unit tests

* Refactor visualize dataset stats from DataNodeMetadata to DataNode (#1499)

* add stats to data node

* lint and format check fix

* fix pytests

* fix layout issue

* fix transcoded data stats

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* initial draft for resolving connection error

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* Support for Python 3.11 (#1502)

* initial draft for python 3.11 support

* update release doc

* add python warnings for e2e tests

* modify e2e test

* modify e2e test

* test by removing lower req scenario

* skip e2e tests for lower bound requirement on python 3.11

* skip e2e tests for lower bound requirement on python 3.11

* remove print statements

---------

Co-authored-by: Nok Lam Chan <nok_lam_chan@mckinsey.com>
Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* Remove Python Upper Bound Requirements  (#1506)

* initial draft for python 3.11 support

* update release doc

* add python warnings for e2e tests

* modify e2e test

* modify e2e test

* test by removing lower req scenario

* skip e2e tests for lower bound requirement on python 3.11

* skip e2e tests for lower bound requirement on python 3.11

* remove python upperbounds initial draft

* fix lint and format errors

* test remove upperbound warning

* test lowerbound pandas install

* revert back pandas requirement

* bump lower requirements for pandas

* remove upper bound clean up

* update release notes

* fix PR comments

---------

Co-authored-by: Nok Lam Chan <nok_lam_chan@mckinsey.com>
Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* refactor launchers and test code

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* modify unit tests

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix lint errors

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* Fix: Adding favicon to Kedro Demo (#1509)

* Fix: Adding favicon to Kedro Demo

* Fix: Change in approach for serving favicon

* Lint error fix

* Lint error fix

* Favicon endpoint test added

* Favicon endpoint test added

* Lint error fixed

* Fix: Adding favicon to Kedro Demo

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Fix: Change in approach for serving favicon

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Lint error fix

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Lint error fix

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Favicon endpoint test added

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Favicon endpoint test added

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Lint error fixed

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Fixed favicon endpoint test

* Release doc updated

* Update RELEASE.md

Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>

* Removed pytest.fixture as per review comment

---------

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>
Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>
Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* fix run_viz tests

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* update unit test for coverage

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* Release v6.5.0 (#1513)

* v6.5.0

* release

* update-reminder-content

* update reminder

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* remove branch condition for automate release version check (#1514)

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* update unit tests

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* add release record

* modify comment

* fix PR comments

* DCO fix

* fixing dco

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

* update pytest

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>

---------

Signed-off-by: ravi-kumar-pilla <ravi_kumar_pilla@mckinsey.com>
Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>
Co-authored-by: Rashida Kanchwala <rashida.kanchwala@quantumblack.com>
Co-authored-by: Nok Lam Chan <nok_lam_chan@mckinsey.com>
Co-authored-by: Jitendra Gundaniya <38945204+jitu5@users.noreply.github.com>
Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>
vladimir-mck added a commit that referenced this pull request Sep 13, 2023
@vladimir-mck @jitu5
@rashidakanchwala @ravi-kumar-pilla @tynandebold
Update dependencies to work with Vite and NextJS, combine css files i…
2d0c577 
…nto a single css file (#1510)

* Update dependencies, update apollo config, update worker to use a mock worker for SSR

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Fix: Adding favicon to Kedro Demo (#1509)

* Fix: Adding favicon to Kedro Demo

* Fix: Change in approach for serving favicon

* Lint error fix

* Lint error fix

* Favicon endpoint test added

* Favicon endpoint test added

* Lint error fixed

* Fix: Adding favicon to Kedro Demo

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Fix: Change in approach for serving favicon

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Lint error fix

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Lint error fix

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Favicon endpoint test added

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Favicon endpoint test added

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Lint error fixed

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>

* Fixed favicon endpoint test

* Release doc updated

* Update RELEASE.md

Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>

* Removed pytest.fixture as per review comment

---------

Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>
Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>
Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Release v6.5.0 (#1513)

* v6.5.0

* release

* update-reminder-content

* update reminder

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* remove branch condition for automate release version check (#1514)

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Update RELEASE.md

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Update css imports to scss, combine css files into 1 for the npm lib, update babel to remove scss imports from the lib components. Revert apollo config and worker

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Update package lock

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Update readme and release me

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Remove build:css script from circle ci config

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Bundle css into lib/styles folder

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Update readme

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Add webpack config to extract scss and babel plugin to remove scss imports

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>

* Update RELEASE.md

Co-authored-by: Tynan DeBold <thdebold@gmail.com>

* Update RELEASE.md

---------

Signed-off-by: Vladimir <vladimir_nikolic@external.mckinsey.com>
Signed-off-by: Jitendra Gundaniya <jitendra_gundaniya@mckinsey.com>
Co-authored-by: Jitendra Gundaniya <38945204+jitu5@users.noreply.github.com>
Co-authored-by: rashidakanchwala <37628668+rashidakanchwala@users.noreply.github.com>
Co-authored-by: Ravi Kumar Pilla <ravi_kumar_pilla@mckinsey.com>
Co-authored-by: Tynan DeBold <thdebold@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jitu5 jitu5 jitu5 approved these changes

@merelcht merelcht merelcht approved these changes

@NeroOkwa NeroOkwa NeroOkwa approved these changes

@ravi-kumar-pilla ravi-kumar-pilla ravi-kumar-pilla approved these changes

@vladimir-mck vladimir-mck vladimir-mck approved these changes

@tynandebold tynandebold Awaiting requested review from tynandebold

@yetudada yetudada Awaiting requested review from yetudada

@astrojuanlu astrojuanlu Awaiting requested review from astrojuanlu astrojuanlu is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@rashidakanchwala @jitu5 @merelcht @NeroOkwa @ravi-kumar-pilla @vladimir-mck