Skip to content

feat: Allow awaiting job.stop()#977

Merged
intcreator merged 2 commits intokelektiv:mainfrom
ehooi:allow-awaiting-stop
Apr 14, 2025
Merged

feat: Allow awaiting job.stop()#977
intcreator merged 2 commits intokelektiv:mainfrom
ehooi:allow-awaiting-stop

Conversation

@ehooi
Copy link
Copy Markdown
Contributor

@ehooi ehooi commented Apr 11, 2025

Description

I believe this would be useful for building a job manager (see: #871)
and supporting graceful shutdown (see: #713).

Related Issue

#976

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • If my change introduces a breaking change, I have added a ! after the type/scope in the title (see the Conventional Commits standard).

@ehooi ehooi changed the title Allow awaiting job.stop() feat: Allow awaiting job.stop() Apr 11, 2025
Copy link
Copy Markdown
Collaborator

@intcreator intcreator left a comment

Choose a reason for hiding this comment

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

haha short diffs are the best diffs

@intcreator intcreator merged commit e296b76 into kelektiv:main Apr 14, 2025
18 checks passed
@intcreator
Copy link
Copy Markdown
Collaborator

closes #976

@ncb000gt
Copy link
Copy Markdown
Member

🎉 This PR is included in version 4.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

intcreator pushed a commit that referenced this pull request Apr 15, 2025
## [4.2.0](v4.1.4...v4.2.0) (2025-04-14)

### ✨ Features

* Allow awaiting job.stop() ([#977](#977)) ([e296b76](e296b76)), closes [#976](#976)

### ♻️ Chores

* **action:** update github/codeql-action action to v3.28.15 ([baf9c7e](baf9c7e))
* **action:** update marocchino/sticky-pull-request-comment action to v2.9.2 ([#975](#975)) ([df57bef](df57bef))
* **action:** update step-security/harden-runner action to v2.11.1 ([da1764d](da1764d))
* **deps:** lock file maintenance ([c37a3ec](c37a3ec))
* **deps:** lock file maintenance ([#979](#979)) ([6a355a3](6a355a3))
* **deps:** pin dependencies ([6ddc31e](6ddc31e))
* **deps:** update dependency [@fast-check](https://github.com/fast-check)/jest to v2.1.1 ([3fa6836](3fa6836))
* **deps:** update dependency [@swc](https://github.com/swc)/core to v1.11.18 ([00d0685](00d0685))
* **deps:** update dependency [@types](https://github.com/types)/node to v22.14.0 ([7dedc90](7dedc90))
* **deps:** update dependency sinon to v19.0.5 ([448934c](448934c))
* **deps:** update dependency sinon to v20 ([#974](#974)) ([ac25eff](ac25eff))
* **deps:** update dependency typescript to v5.8.3 ([7b583a4](7b583a4))
* **deps:** update linters ([#978](#978)) ([cdb638a](cdb638a))
* **deps:** update node.js to v23.11.0 ([#973](#973)) ([7d457cf](7d457cf))
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 this pull request may close these issues.

3 participants