Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

MarcEspiard
Copy link

Adds support for Gitlab CI/CD in getSentryRelease function

@AbhiPrasad
Copy link
Member

Hey! thanks for the PR :)

We'll merge this in soon - let me update https://github.com/getsentry/sentry-javascript-bundler-plugins/ first!

@AbhiPrasad AbhiPrasad self-requested a review June 17, 2024 15:04
@AbhiPrasad AbhiPrasad self-assigned this Jun 17, 2024
@AbhiPrasad
Copy link
Member

Looking at codecov there seems to be more environmental variables that are Gitlab specific! https://github.com/codecov/codecov-cli/blob/24d49dd4a4e879687218faf97cfc6f5b9d83a413/codecov_cli/helpers/ci_adapters/gitlab_ci.py#L14-L16

@MarcEspiard
Copy link
Author

Hey, I only added the commit sha to stay consistent with the rest of the values for the other providers.
I'm pretty sure the commit sha is always there but happy to update with the additional values if needed.
Would we follow the same order as codecov or put the commit sha first and fallback to the other 2?

@MarcEspiard
Copy link
Author

Just saw you opened a PR for a lot more providers and you won't need this, thanks for taking care of this so fast :)

@AbhiPrasad
Copy link
Member

@MarcEspiard I will prob open a new PR and sort out the ordering, I'll just cherry-pick your commit to you get the contribution creds.

Thanks again for the PR, it's a good reminder for us to get this logic working consistently everywhere!

@AbhiPrasad
Copy link
Member

Closing this in favour of #12529 and getsentry/sentry-javascript-bundler-plugins#549

@AbhiPrasad AbhiPrasad closed this Jun 18, 2024
AbhiPrasad added a commit that referenced this pull request Jun 19, 2024
ref #12513

This PR upgrades release detection for Node and Vercel Edge SDKs.

This change adds release name detection for:

- GitLab CI
- AppVeyor
- AWS Amplify
- Azure Pipelines
- Bitrise
- Buddy CI
- Cirrus CI
- Codefresh
- Codemagic
- Semaphore CI
- Travis CI
- CloudBees CodeShip
- Coolify
- Jenkins
- TeamCity
- Woodpecker CI

Based on
getsentry/sentry-javascript-bundler-plugins#549

Supercedes #12507
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.