Skip to content

permitpool: add new module#153

Merged
jefferai merged 1 commit intomainfrom
jbrandhorst-add-permitpool
Jan 22, 2025
Merged

permitpool: add new module#153
jefferai merged 1 commit intomainfrom
jbrandhorst-add-permitpool

Conversation

@johanbrandhorst
Copy link
Copy Markdown
Contributor

The permitpool module contains a single package that implements a pool of permits, allowing limited number of concurrent executions.

Comment thread permitpool/permitpool.go
jefferai
jefferai previously approved these changes Jan 21, 2025
Copy link
Copy Markdown
Member

@jefferai jefferai left a comment

Choose a reason for hiding this comment

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

LGTM but please add package doc as @tmessi suggested

The permitpool module contains a single package that
implements a pool of permits, allowing limited number
of concurrent executions.
@johanbrandhorst johanbrandhorst force-pushed the jbrandhorst-add-permitpool branch from ebf9b3e to 97dd068 Compare January 22, 2025 18:49
@jefferai jefferai merged commit b76c43d into main Jan 22, 2025
@jefferai jefferai deleted the jbrandhorst-add-permitpool branch January 22, 2025 18:53
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.

3 participants