Skip to content

Daniel/cleanup unused code#586

Open
dadukhankevin wants to merge 13 commits intomainfrom
daniel/cleanup_unused_code
Open

Daniel/cleanup unused code#586
dadukhankevin wants to merge 13 commits intomainfrom
daniel/cleanup_unused_code

Conversation

@dadukhankevin
Copy link
Contributor

@dadukhankevin dadukhankevin commented Feb 4, 2026

Removed many unused/old files, combined tiny utilities that go together, removed commented out code blocks, converted large .ts files to json, this is faster and more space efficient.

@dadukhankevin dadukhankevin linked an issue Feb 4, 2026 that may be closed by this pull request
/**
* Cleans a word for spellchecking by removing non-letter characters
*/
function cleanWord(word: string | undefined | null): string {
Copy link
Collaborator

Choose a reason for hiding this comment

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

we don't need this anymore

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@BenjaminScholtens Do we need the file?
(ln 122-123)

                    words.forEach((word: string, idx: number) => {
                        const cleanedWord = cleanWord(word);

because the function itself is used later, but if we don't need the file, we can just delete all of it?

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.

Simplify Codebase

2 participants