Skip to content

Conversation

@Rich-Harris
Copy link
Member

Secrets are redacted from GitHub Actions logs. Since we have a TURBO_TEAM environment variable, to enable remote caching, we have a secret whose value is svelte. That results in some... interesting redactions:

image

Since it's not really a secret that we're the svelte team, this replaces the secret with a hardcoded value.

image

@changeset-bot
Copy link

changeset-bot bot commented Apr 20, 2022

⚠️ No Changeset found

Latest commit: 46de02c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Conduitry
Copy link
Member

Looks good to me, but I don't know how this could be failing linting now ...

https://github.com/sveltejs/kit/runs/6102827202

@Rich-Harris
Copy link
Member Author

The lint error is something unrelated that snuck in. Will be fixed by #4674

@Rich-Harris Rich-Harris merged commit 6b50b12 into master Apr 20, 2022
@Rich-Harris Rich-Harris deleted the too-many-secrets branch April 20, 2022 22:16
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.

4 participants