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: Uses correct generated token when checking out cli repo in release #44

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

jpmcb
Copy link
Member

@jpmcb jpmcb commented Sep 27, 2023

Description

Fixes missing generated token during CI release action.

Includes some whitespace fixes.

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

Related to #35 (comment) (cc @Anush008)

As a result, it defaults to the default GITHUB_TOKEN credential. I think this is causing an issue with the @semantic-release/git plugin, as this credential does not have the required permission to operate on protected branches.

Mobile & Desktop Screenshots/Recordings

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 README.md
  • 📓 docs.opensauced.pizza
  • 🍕 dev.to/opensauced
  • 📕 storybook
  • 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

fix: correct spacing in .github/workflows

Signed-off-by: John McBride <[email protected]>
@jpmcb jpmcb requested review from Anush008 and a team September 27, 2023 21:21
Copy link
Member

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@jpmcb jpmcb merged commit 1e0c9f1 into open-sauced:beta Sep 27, 2023
@jpmcb jpmcb deleted the update-ci-release branch September 27, 2023 21:38
open-sauced bot pushed a commit that referenced this pull request Sep 27, 2023
## [1.0.0-beta.6](v1.0.0-beta.5...v1.0.0-beta.6) (2023-09-27)

### 🐛 Bug Fixes

* Uses correct generated token when checking out cli repo in release ([#44](#44)) ([1e0c9f1](1e0c9f1))
open-sauced bot pushed a commit that referenced this pull request Oct 11, 2023
## 1.0.0 (2023-10-11)

### 🤖 Build System

* sematic bin release, npm ([7b4607e](7b4607e))

### 🔁 Continuous Integration

* Update @open-sauced/[email protected] and compliance.yaml ([#33](#33)) ([146b6b7](146b6b7))

### 🐛 Bug Fixes

* Uses correct generated token when checking out cli repo in release ([#44](#44)) ([1e0c9f1](1e0c9f1))

### 🍕 Features

* Add install instructions and script for pizza CLI ([#26](#26)) ([421a429](421a429))
* Add posthog telemetry integration ([#37](#37)) ([9829f49](9829f49))
* cli auth ([#21](#21)) ([34728fb](34728fb))
* GitHub action to build and upload Go artifacts after release created ([#22](#22)) ([ad187a9](ad187a9))
* Http Client for accessing OpenSauced API client ([#23](#23)) ([ec2b357](ec2b357))
* Leverage the GITHUB_APP_TOKEN for releases ([#32](#32)) ([e0a25e0](e0a25e0))
* npm i -g pizza ([73291d1](73291d1))
* Pizza show ([#24](#24)) ([72f21ce](72f21ce))
* provide repository contributors insights ([#30](#30)) ([d16091f](d16091f))
* provide repository insights ([#38](#38)) ([dc148d6](dc148d6))
* repo-query support ([199cfd7](199cfd7))
* update bin name release.yaml ([6b21cb8](6b21cb8))
* Version command for CLI based on release builds ([#36](#36)) ([9f3eedc](9f3eedc))
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.

3 participants