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.

Sign up for GitHub

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

Jump to bottom

Use a job for a pre-publishing #55

Merged
merged 2 commits into from Aug 18, 2021
Merged

Use a job for a pre-publishing #55

merged 2 commits into from Aug 18, 2021

Conversation

madnificent
Copy link
Member

The prepublisher con now emit jobs for lengthy operations.

Options:

  • We might want to extract the job route and handling code into a separate file
  • We may want to add this for the notulen too

@madnificent
Use a job for a pre-publishing
f3d064d 
The prepublisher con now emit jobs for lengthy operations.

Pairing with @abeforgit
@madnificent
Fix linting errors
22f334d 
Unused import was removed.
@abeforgit abeforgit added bug Something isn't working enhancement New feature or request labels Aug 18, 2021
@abeforgit abeforgit merged commit f043975 into master Aug 18, 2021
@abeforgit abeforgit deleted the feature/prepublish-with-job branch August 18, 2021 11:08
@abeforgit abeforgit mentioned this pull request Aug 18, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@madnificent @abeforgit