Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

ref(cogs): Remove unused cogs enabled flag #4060

Merged
merged 3 commits into from
Sep 26, 2024
Merged

ref(cogs): Remove unused cogs enabled flag #4060

merged 3 commits into from
Sep 26, 2024

Conversation

Dav1dde
Copy link
Member

@Dav1dde Dav1dde commented Sep 23, 2024

After the Kafka removal for COGS this is now unused.

#skip-changelog

@Dav1dde Dav1dde requested a review from a team as a code owner September 23, 2024 07:56
@Dav1dde Dav1dde self-assigned this Sep 23, 2024
jjbayer
jjbayer approved these changes Sep 23, 2024
View reviewed changes
Copy link
Member

@jjbayer jjbayer left a comment

Choose a reason for hiding this comment

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

Could you add a changelog entry for this?

@Dav1dde Dav1dde enabled auto-merge (squash) September 23, 2024 13:59
@Dav1dde Dav1dde merged commit cfa0358 into master Sep 26, 2024
23 checks passed
@Dav1dde Dav1dde deleted the dav1d/cogs-enabled-rm branch September 26, 2024 15:34
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jjbayer jjbayer jjbayer approved these changes

Assignees

@Dav1dde Dav1dde

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Dav1dde @jjbayer