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

Revert "[convert] Ensure not supported lifecycle hook warning before … #1810

Merged
merged 1 commit into from
Mar 29, 2024

Conversation

VenelinMartinov
Copy link
Contributor

…an error"

This reverts commit 1c48a51.

Will get it in after releasing the bridge since downstream tests were run without this commit.

Copy link

codecov bot commented Mar 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.08%. Comparing base (fb81f18) to head (9b93d08).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1810      +/-   ##
==========================================
- Coverage   60.62%   60.08%   -0.54%     
==========================================
  Files         303      310       +7     
  Lines       42353    42726     +373     
==========================================
- Hits        25675    25671       -4     
- Misses      15206    15583     +377     
  Partials     1472     1472              

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

@VenelinMartinov VenelinMartinov merged commit 67f7871 into master Mar 29, 2024
9 checks passed
@VenelinMartinov VenelinMartinov deleted the vvm/revert_1803 branch March 29, 2024 15:34
VenelinMartinov added a commit that referenced this pull request Mar 29, 2024
Zaid-Ajaj added a commit that referenced this pull request Mar 29, 2024
#1811)

… before … (#1810)"

This reverts commit 67f7871 and gets
#1803 back in.

I originally reverted it since it was merged after the commit we
pre-tested the bridge upgrade on. Merging it back in
VenelinMartinov added a commit that referenced this pull request Mar 29, 2024
VenelinMartinov added a commit that referenced this pull request Mar 29, 2024
#1815)

… warning before … (#1810)""

This reverts commit 24076f1.

Reverting the PR again, it got merged before the pf release.

Lovely bridge release today.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants