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

chore: restore deprecate as an internal module #40124

Merged
merged 2 commits into from
Oct 8, 2023

Conversation

miniak
Copy link
Contributor

@miniak miniak commented Oct 6, 2023

Revert #35311 as it prevents us from testing deprecation warnings as the tests contain a different copy of the deprecate module than Electron itself

Also add missing tests for deprecated event warnings to unblock #40089

Notes: none

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Oct 6, 2023
@miniak miniak requested a review from dsanders11 October 6, 2023 09:48
@miniak miniak self-assigned this Oct 6, 2023
@miniak miniak added semver/patch backwards-compatible bug fixes no-backport labels Oct 6, 2023
@miniak miniak force-pushed the miniak/restore-deprecate-module branch from 872bb65 to 8d36be2 Compare October 6, 2023 09:52
@miniak miniak force-pushed the miniak/restore-deprecate-module branch 3 times, most recently from eaf4ea0 to 1070697 Compare October 6, 2023 10:14
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Oct 7, 2023
@miniak miniak force-pushed the miniak/restore-deprecate-module branch from 1070697 to 2de66c4 Compare October 7, 2023 11:23
@miniak miniak force-pushed the miniak/restore-deprecate-module branch from 2de66c4 to 9d09ef0 Compare October 7, 2023 11:26
@miniak miniak requested a review from zcbenz October 8, 2023 09:44
@zcbenz zcbenz merged commit 737e3de into main Oct 8, 2023
15 checks passed
@zcbenz zcbenz deleted the miniak/restore-deprecate-module branch October 8, 2023 23:55
@release-clerk
Copy link

release-clerk bot commented Oct 8, 2023

No Release Notes

@miniak
Copy link
Contributor Author

miniak commented Oct 9, 2023

/trop run backport-to 27-x-y

@trop
Copy link
Contributor

trop bot commented Oct 9, 2023

The backport process for this PR has been manually initiated - sending your PR to 27-x-y!

@trop
Copy link
Contributor

trop bot commented Oct 9, 2023

I was unable to backport this PR to "27-x-y" cleanly;
you will need to perform this backport manually.

miniak added a commit that referenced this pull request Oct 9, 2023
miniak added a commit that referenced this pull request Oct 9, 2023
jkleinsc pushed a commit that referenced this pull request Oct 10, 2023
Revert "chore: restore deprecate as an internal module (#40124)"

This reverts commit 737e3de.
MrHuangJser pushed a commit to MrHuangJser/electron that referenced this pull request Dec 11, 2023
* Revert "refactor: don't expose deprecate as an internal module (electron#35311)"

This reverts commit 8424779.

* check crashed event warnings
MrHuangJser pushed a commit to MrHuangJser/electron that referenced this pull request Dec 11, 2023
Revert "chore: restore deprecate as an internal module (electron#40124)"

This reverts commit 737e3de.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-backport semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants