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

Incorrect buildkite slug #333

Closed
vio opened this issue Dec 26, 2023 · 2 comments · Fixed by #334
Closed

Incorrect buildkite slug #333

vio opened this issue Dec 26, 2023 · 2 comments · Fixed by #334
Labels

Comments

@vio
Copy link
Contributor

vio commented Dec 26, 2023

Bug

The slug value on buildkite has an incorrect format: ORG/ORG/PROJECT. Both ORG and PROJECT values refer to the buildkite specific configuration and can differ from the repository slug.

  relative-ci:agent env-ci environment variables {
  isCi: true,
  name: 'Buildkite',
  service: 'buildkite',
  build: '17',
  buildUrl: 'https://buildkite.com/rltv/example/builds/17',
  commit: '38649ff7dc00526a539a4e0759d25fbe8c7d537f',
  tag: '',
  branch: 'main',
  slug: 'rltv/rltv/example',
  pr: '1',
  isPr: true,
  prBranch: 'update-agent',
  root: '/opt/homebrew/var/buildkite-agent/builds/mbp22-local-1/rltv/example'

Documentation: https://buildkite.com/docs/pipelines/environment-variables#buildkite-environment-variables

buildkite does not provide a specific environment variable for the actual repository slug, but provides the full git repo URL: BUILDKITE_REPO. A similar method as for #249 (comment) can be used to extract the real repository slug.

@travi
Copy link
Member

travi commented Dec 26, 2023

PRs welcome

@vio vio mentioned this issue Dec 27, 2023
@travi travi closed this as completed in #334 Jan 6, 2024
Copy link

🎉 This issue has been resolved in version 11.1.0 🎉

The release is available on:

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 a pull request may close this issue.

2 participants