Skip to content

Update good_job to include bug fix for polling feature#9643

Merged
mitchellhenke merged 1 commit intomainfrom
mitchellhenke/update-good-job-4
Nov 24, 2023
Merged

Update good_job to include bug fix for polling feature#9643
mitchellhenke merged 1 commit intomainfrom
mitchellhenke/update-good-job-4

Conversation

@mitchellhenke
Copy link
Contributor

🛠 Summary of changes

bensheldon/good_job#1148 includes a fix to a bug with a feature/configuration that we use. This PR updates good_job to a version that includes the patch.

@mitchellhenke mitchellhenke requested a review from a team November 22, 2023 15:32
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

other than base64, LGTM

Gemfile.lock Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

same comment as here, I'm wary of upgrading base64 at this time

#9644 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the base64 update from this PR

changelog: Internal, Dependencies, Update good_job
@mitchellhenke mitchellhenke force-pushed the mitchellhenke/update-good-job-4 branch from e67ea41 to 380af53 Compare November 24, 2023 14:52
@mitchellhenke mitchellhenke merged commit f63b646 into main Nov 24, 2023
@mitchellhenke mitchellhenke deleted the mitchellhenke/update-good-job-4 branch November 24, 2023 16:08
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