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

ci: trim branch name for CF preview URLs #289

Merged
merged 1 commit into from
Mar 13, 2024

Conversation

davidlj95
Copy link
Owner

@davidlj95 davidlj95 commented Mar 13, 2024

After checking out a deploy preview, detected that if branch name is too long it gets cut when generating the preview URL.

In one case it was cut to 28 chars but unsure if 28 is the hard limit or if they remove parts (separated by -)
In another case it was also cut to 28 chars so trying with 28 as magic number for now

Worked for this PR:
https://stacked-ci-shorten-cf-previe.davidlj95.pages.dev

🎉

Copy link
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on Graphite Graphite

@davidlj95 davidlj95 changed the title ci: replace slash with dash for CF preview URLs ci: trim branch name for CF preview URLs Mar 13, 2024
@davidlj95 davidlj95 force-pushed the stacked/ci-shorten-cf-preview-url branch from 3f981e9 to cf13cbc Compare March 13, 2024 13:24
@davidlj95 davidlj95 merged commit 78629c3 into main Mar 13, 2024
9 checks passed
@davidlj95 davidlj95 deleted the stacked/ci-shorten-cf-preview-url branch March 13, 2024 13:27
Copy link

🎉 This PR is included in version 2.35.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant