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

Remove unused expression helpers #361

Merged

dbanck merged 1 commit into main from c-remove-unused-helpers

Jan 9, 2024

Member

dbanck commented Jan 8, 2024

In #354, I missed some helper functions for the old expression logic


          Remove unused expression helpers

470a080

dbanck added the technical-debt label

dbanck self-assigned this

dbanck requested a review from a team as a code owner

January 8, 2024 16:25

radeksimko approved these changes

View reviewed changes

Member

radeksimko left a comment

Good catch, it would be nice to have some dead code detector in the pipeline some day 😅

dbanck merged commit 3bd46be into main

5 checks passed

dbanck deleted the c-remove-unused-helpers branch

January 9, 2024 10:00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels