Skip to content

rm unused token#327

Merged
zzstoatzz merged 4 commits intomainfrom
zzstoatzz-patch-1
Jan 4, 2024
Merged

rm unused token#327
zzstoatzz merged 4 commits intomainfrom
zzstoatzz-patch-1

Conversation

@zzstoatzz
Copy link
Copy Markdown
Collaborator

No description provided.

@zzstoatzz zzstoatzz requested a review from a team as a code owner December 13, 2023 20:40
@urimandujano
Copy link
Copy Markdown

Hey zzstoatzz, what's this file used for? If it's for doc purposes, maybe it would make sense to keep the access_token key and set it to a dummy value. If it's configuration for deployment that gets run, I think removing it is the cleanest thing to do.

@zzstoatzz
Copy link
Copy Markdown
Collaborator Author

@urimandujano its for a real deployment, the access token is not necessary since the repo is public

@zzstoatzz zzstoatzz merged commit 35d0a3f into main Jan 4, 2024
@zzstoatzz zzstoatzz deleted the zzstoatzz-patch-1 branch January 4, 2024 22:59
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