Skip to content

Web: Remove old files and dependencies#2801

Merged
wslulciuc merged 2 commits into
mainfrom
remove/old-deps
Apr 22, 2024
Merged

Web: Remove old files and dependencies#2801
wslulciuc merged 2 commits into
mainfrom
remove/old-deps

Conversation

@phixMe

@phixMe phixMe commented Apr 22, 2024

Copy link
Copy Markdown
Member

Problem

Removes files and libraries mostly from the rendering of the old lineage graph.

One-line summary: Drops deps, and removes unused React components.

Checklist

  • You've signed-off your work
  • Your changes are accompanied by tests (if relevant)
  • Your change contains a small diff and is self-contained
  • You've updated any relevant documentation (if relevant)
  • You've included a one-line summary of your change for the CHANGELOG.md (Depending on the change, this may not be necessary).
  • You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
  • You've included a header in any source code files (if relevant)

@boring-cyborg boring-cyborg Bot added the web label Apr 22, 2024
@netlify

netlify Bot commented Apr 22, 2024

Copy link
Copy Markdown

Deploy Preview for peppy-sprite-186812 canceled.

Name Link
🔨 Latest commit 2928667
🔍 Latest deploy log https://app.netlify.com/sites/peppy-sprite-186812/deploys/6625aab0cebea200084a8154

@codecov

codecov Bot commented Apr 22, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.47%. Comparing base (d6ac3e6) to head (2928667).

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2801       +/-   ##
===========================================
+ Coverage        0   84.47%   +84.47%     
- Complexity      0     1429     +1429     
===========================================
  Files           0      251      +251     
  Lines           0     6460     +6460     
  Branches        0      299      +299     
===========================================
+ Hits            0     5457     +5457     
- Misses          0      850      +850     
- Partials        0      153      +153     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@phixMe phixMe marked this pull request as ready for review April 22, 2024 01:08

@wslulciuc wslulciuc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🧹 📦

@wslulciuc wslulciuc merged commit c553401 into main Apr 22, 2024
@wslulciuc wslulciuc deleted the remove/old-deps branch April 22, 2024 23:08
jonathanpmoraes referenced this pull request in nubank/NuMarquez Feb 6, 2025
* Removing old unused files and dependencies.

* Removing old test.

---------

Co-authored-by: phix <peter.hicks@astronomer.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants