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

Refactor and fix spelling issues and errors #5899

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

strmfos
Copy link

@strmfos strmfos commented Mar 19, 2025

Hi team! I fixed all the errors I found
If there's anything else I can help with, I'm happy to do so

apps/circuit-compiler/src/app/components/container.tsx
showCircumCompilerTooltip - showCircomCompilerTooltip

apps/remix-ide-e2e/src/commands/addLocalPlugin.ts
fix missing quote and parenthesis in the .click() call.

apps/remix-ide-e2e/src/commands/checkTerminalFilter.ts
deos - does

apps/remix-ide-e2e/src/commands/connectToExternalHttpProvider.ts
modalDialogCustomPromp - modalDialogCustomPrompt x2

apps/remix-ide-e2e/src/commands/getInstalledPlugins.ts
Added missing closing parenthesis.

apps/remix-ide-e2e/src/commands/hideMetaMaskPopup.ts
Removed unnecessary parentheses from the done call inside browser.execute.

apps/remix-ide-e2e/src/commands/openFile.ts
Added missing closing parenthesis ]

apps/remix-ide-e2e/src/commands/validateValueInput.ts
valueTosSet - valueToSet

apps/remix-ide-e2e/src/local-plugin/src/app/app.tsx
filSystemProfile - fileSystemProfile 2x

apps/remix-ide-e2e/src/tests/code_format.test.ts
unfomattedContract - unformattedContract 2x

apps/remix-ide-e2e/src/tests/dgit_github.test.ts
gitubUsername - githubUsername 2x

Copy link

netlify bot commented Mar 19, 2025

👷 Deploy request for remixproject pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 4f91fd9

@strmfos
Copy link
Author

strmfos commented Mar 27, 2025

@Aniket-Engg should I fix something?

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.

1 participant