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: remove ending slash upload-index-json #34125

Merged
merged 1 commit into from May 6, 2022

Conversation

VerteDinde
Copy link
Member

Description of Change

Follow up to #34104, corrects the path for headers/dist artifacts in upload-index-json.py (specific error here)

Checklist

Release Notes

Notes: no-notes

@VerteDinde VerteDinde added fast-track 🚅 Indicates that this PR is intended to bypass the 24 hour rule. Needs approval from Releases semver/none no-backport labels May 6, 2022
@VerteDinde VerteDinde requested a review from a team as a code owner May 6, 2022 15:28
@MarshallOfSound MarshallOfSound merged commit 2900bc5 into main May 6, 2022
@MarshallOfSound MarshallOfSound deleted the s3-path-correction branch May 6, 2022 19:57
@release-clerk
Copy link

release-clerk bot commented May 6, 2022

No Release Notes

VerteDinde added a commit that referenced this pull request May 7, 2022
* build: remove S3 uploads

* build: remove ending slash upload-index-json (#34125)

Co-authored-by: Keeley Hammond <khammond@slack-corp.com>
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
VerteDinde added a commit that referenced this pull request May 7, 2022
* build: remove S3 uploads

* build: remove ending slash upload-index-json (#34125)

Co-authored-by: Keeley Hammond <khammond@slack-corp.com>
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
VerteDinde added a commit that referenced this pull request May 7, 2022
* build: remove S3 uploads

* build: remove ending slash upload-index-json (#34125)

Co-authored-by: Keeley Hammond <khammond@slack-corp.com>
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
MarshallOfSound pushed a commit that referenced this pull request May 7, 2022
* build: remove S3 uploads

* build: remove ending slash upload-index-json (#34125)

Co-authored-by: Keeley Hammond <khammond@slack-corp.com>
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
VerteDinde added a commit that referenced this pull request May 7, 2022
* build: remove S3 uploads

* build: remove ending slash upload-index-json (#34125)

Co-authored-by: Keeley Hammond <khammond@slack-corp.com>
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fast-track 🚅 Indicates that this PR is intended to bypass the 24 hour rule. Needs approval from Releases no-backport semver/none
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants