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

Remove debug statement in periodic job enqueuer #176

Merged
merged 1 commit into from
Jan 26, 2024

Conversation

brandur
Copy link
Contributor

@brandur brandur commented Jan 26, 2024

This one's my fault -- left a debug statement in #168 as part of the
periodic job enqueuer. Here, remove it.

Also remove a similar, unused commented out debug statement in
client_test.go.

@brandur brandur force-pushed the brandur-remove-debug-statement branch 2 times, most recently from e863b7a to 7331b59 Compare January 26, 2024 01:53
@brandur brandur requested a review from bgentry January 26, 2024 01:57
@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

## [0.0.18] - 2024-01-25
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Went ahead and tagged a new release. Gonna send out a hotfix for this one.

@brandur brandur force-pushed the brandur-remove-debug-statement branch from 7331b59 to 4c52daf Compare January 26, 2024 01:58
This one's my fault -- left a debug statement in #168 as part of the
periodic job enqueuer. Here, remove it.

Also remove a similar, unused commented out debug statement in
`client_test.go`.
@brandur brandur force-pushed the brandur-remove-debug-statement branch from 4c52daf to 14fb4ef Compare January 26, 2024 01:58
Copy link
Contributor

@bgentry bgentry left a comment

Choose a reason for hiding this comment

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

Ugh, I should have caught that 🤦‍♂️

@brandur
Copy link
Contributor Author

brandur commented Jan 26, 2024

TY.

@brandur brandur merged commit 859c662 into master Jan 26, 2024
7 checks passed
@brandur brandur deleted the brandur-remove-debug-statement branch January 26, 2024 14:20
brandur added a commit that referenced this pull request Jan 26, 2024
A small release containing only the fix from #176.
brandur added a commit that referenced this pull request Jan 26, 2024
A small release containing only the fix from #176.
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