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

build: fix version check in release_dependency_versions.yml #37155

Merged
merged 2 commits into from Feb 7, 2023

Conversation

dsanders11
Copy link
Member

Description of Change

Turns out \d isn't supported in POSIX regexes - could use [:digit:] but personally I just prefer [0-9] here.

Example of skipped dependencies for v23.0.0: https://github.com/electron/electron/actions/runs/4109277828/jobs/7090932840

Checklist

  • PR description included and stakeholders cc'd

Release Notes

Notes: none

@jkleinsc
Copy link
Contributor

jkleinsc commented Feb 7, 2023

Merging as CI failure unrelated to PR change.

@jkleinsc jkleinsc merged commit 7abe38b into main Feb 7, 2023
2 checks passed
@jkleinsc jkleinsc deleted the version-check-release-dependencies branch February 7, 2023 16:38
@release-clerk
Copy link

release-clerk bot commented Feb 7, 2023

No Release Notes

khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
…#37155)

* build: fix version check in release_dependency_versions.yml

* chore: also mksnapshot
@dsanders11 dsanders11 added target/22-x-y PR should also be added to the "22-x-y" branch. target/23-x-y PR should also be added to the "23-x-y" branch. and removed no-backport labels Feb 24, 2023
@dsanders11
Copy link
Member Author

/trop run backport-to 22-x-y

@trop
Copy link
Contributor

trop bot commented Feb 24, 2023

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

@trop
Copy link
Contributor

trop bot commented Feb 24, 2023

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

@trop trop bot added needs-manual-bp/22-x-y and removed target/22-x-y PR should also be added to the "22-x-y" branch. labels Feb 24, 2023
@dsanders11
Copy link
Member Author

/trop run backport-to 23-x-y

@trop
Copy link
Contributor

trop bot commented Feb 24, 2023

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

@trop
Copy link
Contributor

trop bot commented Feb 24, 2023

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

@trop trop bot added needs-manual-bp/23-x-y and removed target/23-x-y PR should also be added to the "23-x-y" branch. labels Feb 24, 2023
gecko19 pushed a commit to brightsign/electron that referenced this pull request Feb 28, 2023
…#37155)

* build: fix version check in release_dependency_versions.yml

* chore: also mksnapshot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants