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

fix: release workflow trigger & others #2369

Merged
merged 15 commits into from May 31, 2023

Conversation

ryandagg
Copy link
Contributor

@ryandagg ryandagg commented May 30, 2023

There is a bug(feature?) in GHA on: pull_request that leads to it not always running on closed pull requests.
Refactor various triggers, inputs and entry points to ensure safeguards in appropriate places.
Attempt to manually log in to GH cli by running gh auth login --with-token with GITHUB_TOKEN
Fix error in release-homebrew.yml. Multiple environment secrets can not be used in 1 job. Moved S3 & CloudFront secrets to repo level.

@ryandagg ryandagg requested a review from a team as a code owner May 30, 2023 18:51
k80bowman
k80bowman previously approved these changes May 30, 2023
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:26 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:26 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:26 — with GitHub Actions Inactive
@ryandagg ryandagg dismissed k80bowman’s stale review May 30, 2023 21:28

Significant changes since review.

@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:46 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:46 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:46 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:52 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:52 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:52 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:55 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:55 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 21:55 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 22:00 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 22:00 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 22:00 — with GitHub Actions Inactive
…instead of separate file

separate publish npm & release GH into workflows and better name npm scripts
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 23:25 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 23:25 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 23:25 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 23:33 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 23:33 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 30, 2023 23:33 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 15:47 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 15:48 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 15:48 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 15:59 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 15:59 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 15:59 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 16:00 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 16:00 — with GitHub Actions Inactive
@ryandagg ryandagg temporarily deployed to AcceptanceTests May 31, 2023 16:00 — with GitHub Actions Inactive
@ryandagg ryandagg merged commit 2b4a95c into main May 31, 2023
11 checks passed
@ryandagg ryandagg deleted the rd/cd/fix-release-workflow-trigger branch May 31, 2023 16:15
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