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.

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

Address all yarn test warnings #24722

Merged
merged 1 commit into from
Jun 29, 2022

Conversation

bbovenzi
Copy link
Contributor

@bbovenzi bbovenzi commented Jun 29, 2022

  • remove outdated react-hooks library (removes some test and install warnings)
  • add missing mock

^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragement file, named {pr_number}.significant.rst, in newsfragments.

- remove outdated react-hooks library
- add missing mock
@boring-cyborg boring-cyborg bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Jun 29, 2022
@bbovenzi bbovenzi marked this pull request as ready for review June 29, 2022 02:16
@bbovenzi bbovenzi added this to the Airflow 2.3.3 milestone Jun 29, 2022
@bbovenzi bbovenzi merged commit c15cf6e into apache:main Jun 29, 2022
@bbovenzi bbovenzi deleted the fix-react-testing-warnings branch June 29, 2022 03:06
ephraimbuddy pushed a commit that referenced this pull request Jun 30, 2022
- remove outdated react-hooks library
- add missing mock

(cherry picked from commit c15cf6e)
@ephraimbuddy ephraimbuddy added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Jun 30, 2022
@ephraimbuddy ephraimbuddy added type:misc/internal Changelog: Misc changes that should appear in change log and removed changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) labels Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues type:misc/internal Changelog: Misc changes that should appear in change log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants