Skip to content

Comments

Alias the required GITHUB_PACKAGES_TOKEN during dotfiles setup#25

Open
ian-frost wants to merge 1 commit intomainfrom
nt/set-both-github-token-vars-needed
Open

Alias the required GITHUB_PACKAGES_TOKEN during dotfiles setup#25
ian-frost wants to merge 1 commit intomainfrom
nt/set-both-github-token-vars-needed

Conversation

@ian-frost
Copy link

GITHUB_TOKEN is required for Github CLI but our applications require the PAT to be set to GITHUB_PACKAGES_TOKEN this PR sets up both from the same value for a new starter

I don't quite understand why there is no export keyword in front of this but choosing to follow the convention here assuming the environment scripts require this? Feedback welcome 😂

@ian-frost ian-frost requested a review from micahvdk February 10, 2026 02:05
Copy link
Contributor

@micahvdk micahvdk left a comment

Choose a reason for hiding this comment

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

LGTM

export is just a more explicit way in making it an environment variable

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.

2 participants