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

Move slack adapter to separate package #337

Merged
merged 1 commit into from
Aug 8, 2024
Merged

Move slack adapter to separate package #337

merged 1 commit into from
Aug 8, 2024

Conversation

timkpaine
Copy link
Member

@timkpaine timkpaine commented Jul 16, 2024 •
edited
Loading

As per title, moves the slack adapter to csp-adapter-slack package

@timkpaine timkpaine added tag: internal Issues and PRs for maintainance of the project - not interesting to external users adapter: slack Issues and PRs related to our Slack adapter labels Jul 16, 2024
@timkpaine timkpaine marked this pull request as ready for review July 16, 2024 20:03
robambalu
robambalu previously approved these changes Aug 6, 2024
View reviewed changes
@timkpaine timkpaine merged commit cc77b7d into main Aug 8, 2024
30 checks passed
@timkpaine timkpaine deleted the tkp/rmslack branch August 8, 2024 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robambalu robambalu robambalu approved these changes

@AdamGlustein AdamGlustein Awaiting requested review from AdamGlustein AdamGlustein is a code owner

@svatasoiu svatasoiu Awaiting requested review from svatasoiu svatasoiu is a code owner

@alexddobkin alexddobkin Awaiting requested review from alexddobkin alexddobkin is a code owner

@ptomecek ptomecek Awaiting requested review from ptomecek ptomecek is a code owner

Assignees
No one assigned
Labels
adapter: slack Issues and PRs related to our Slack adapter tag: internal Issues and PRs for maintainance of the project - not interesting to external users
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@timkpaine @robambalu