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

replaced repeated lines with a function varReplace() #16

Open

Ray0716 wants to merge 1 commit into kokonut27:main from Ray0716:main

Contributor

Ray0716 commented Dec 31, 2023

it seemed like you had a lot of repeated lines where you're replacing strings in strings. i cleaned up your code so it uses a function instead


          replaced repeated lines with a function replaceVal()

2327c70

Owner

kokonut27 commented Jan 1, 2024

quick question, looks like it's fine, but have you tried testing it out?

i think i would suggest adding a return var at the very end, but i can add that on my own lol.

kokonut27 assigned kokonut27 and Ray0716

kokonut27 added the enhancement label

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