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

feat: Automatic Release detection (#335) #363

Merged
merged 1 commit into from
Jun 10, 2021

Conversation

AramisHM
Copy link
Contributor

@AramisHM AramisHM commented May 31, 2021

Attempts to guess the release info, either by getting the last git
commit hash or by looking up some know environment variables that tracks
release identification.

Had to make changes on some tests, though I am not sure if what I wrote there is appropriated.

Closes #335.

@bruno-garcia bruno-garcia requested a review from rhcarvalho June 1, 2021 18:22
@AramisHM AramisHM force-pushed the automatic-release-detection branch 2 times, most recently from a972959 to 5d9e534 Compare June 1, 2021 21:15
Copy link

@messer messer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If so inclined, please see the note I left on the GetDefaultRelease() function declaration.

util.go Outdated Show resolved Hide resolved
Copy link
Contributor

@rhcarvalho rhcarvalho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AramisHM thanks for your contribution, muito obrigado! Only required change is to not introduce any new exported symbol. We can always revisit how exactly the automatic detection works later.

util.go Outdated Show resolved Hide resolved
http/sentryhttp_test.go Outdated Show resolved Hide resolved
util.go Outdated Show resolved Hide resolved
util.go Outdated Show resolved Hide resolved
Attempts to guess the release info, either by getting the last git
commit hash or by looking up some known environment variables that tracks
release identification.
@AramisHM AramisHM force-pushed the automatic-release-detection branch from 5d9e534 to 790558d Compare June 8, 2021 19:28
@AramisHM
Copy link
Contributor Author

AramisHM commented Jun 8, 2021

Thank you for taking the time to deal with a newbie.

Wasn't sure if create a new commit, ended up force pushing the new code on top of the old one.

@rhcarvalho rhcarvalho linked an issue Jun 10, 2021 that may be closed by this pull request
Copy link
Contributor

@rhcarvalho rhcarvalho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @AramisHM!

@rhcarvalho rhcarvalho merged commit 4f72d77 into getsentry:master Jun 10, 2021
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.

Automatic Release detection
3 participants