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.

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

Converted src/languages.js from JS to TS #95

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alanouda2
Copy link

@alanouda2 alanouda2 commented Aug 30, 2023

fixed #43
I updated the type declarations for all variables, functions, and objects to ensure they are no longer of type 'any.' To validate the correctness of the TypeScript file, I executed 'npm run lint' and 'npm run test' tests.

@alanouda2
Copy link
Author

This only has one commit as I reinstalled NobeBB and inserted my code. My file runs on my local computer (as I showed the Professor) but it does not pass the git hub tests. I can't try and fix the errors locally when I can not see the errors and I don't have access to re-run the tests on Github. I have worked really hard translating this file as I do not know Typescript or Javascript and I am sure my translation is right as I ran it locally with no errors and high coverage.

@alanouda2
Copy link
Author

git

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.

Converting src/languages.js from JS to TS
1 participant