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

Run jshint and clean up issues #867

Merged
merged 1 commit into from
Feb 1, 2025
Merged

Run jshint and clean up issues #867

merged 1 commit into from
Feb 1, 2025

Conversation

TimRudy
Copy link
Contributor

@TimRudy TimRudy commented Feb 1, 2025

There are code changes in 2 files:

  • app\scripts\services\drivers.js - moved functions inside ng scope
  • app\scripts\graphics\joint.routers.js - changed != null to !== null

@cavearr
Copy link
Member

cavearr commented Feb 1, 2025

Wooo what a rhythm! we go like a rocket!

@cavearr cavearr merged commit 5b73fa6 into FPGAwars:develop Feb 1, 2025
4 of 6 checks passed
@cavearr
Copy link
Member

cavearr commented Feb 1, 2025

@TimRudy tell me when you do for finish this first start point and i'll start the automatic prettier in actions.

Good work!!

@TimRudy
Copy link
Contributor Author

TimRudy commented Feb 1, 2025 via email

@cavearr
Copy link
Member

cavearr commented Feb 1, 2025

@TimRudy, take your time! There is no rush. I'm only telling you this because I don't want to activate the bot until you've finished the first full batch. I'll check the package-lock.json, don't worry 👍
As I said, don't get overwhelmed with this and at your own pace.

@TimRudy
Copy link
Contributor Author

TimRudy commented Feb 3, 2025 via email

@cavearr
Copy link
Member

cavearr commented Feb 3, 2025

Hi @TimRudy ! i don't have @lmcapacho mail, @lmcapacho if you want write tim and me an email (in fpgawars list are our mails ) to meet in contact!

In the afternoon i'll reply you, the sdk is disabled in the package, i'll build the next wip with the suppor of sdk, sorry Tim! but if you start from source, change in package.json the nwjs package from 0.95.0 to "nw": "0.95.0-sdk" and reinstall it. And upgrade your nodejs to 23.3.0 (i updated nwjs this weekend) with that you have F12 again if you launch from source

@cavearr
Copy link
Member

cavearr commented Feb 3, 2025

Hi @TimRudy ! here you are!:

1. I suppose we should change these to @v4 in all the .yml files

This is on my side, when you finish the first "manual check" i activate the bot account and start to prettify automatically, tell me when you do for finished the manual check.

2. Would you like to give me Spanish for these?

"Are you sure you want to clear the recent projects list?"
¿Desea vaciar el listado de proyectos recientes?

"Clear Recent Projects"
Vaciar listado de proyectos recientes

"Clear Recently Opened"
Vaciar ficheros abiertos recientemente.

"Don’t Save"
No guardes

"Error enabling driver type {{type}}" <- (type==ftdi or serial)
Error habilitando driver clase {{type}}

"Error reading the file: {{error}}"
Error abriendo el fichero: {{error}}

"Import code" <- Should we say Import source....
Importar código. <- I like your suggestion and i like "import source code", in near future we could import other languages

"Open Recent"
Abierto recientemente

"Recent projects cleared"
Proyectos recientes vaciado.

"Screen recording error. Review your permissions."
Error al grabar, puede ser un error de permisos, revisa la configuración.

"Snapshot saved: {{name}}"
Imagen guardada:{{name}}

"Video saved: {{name}}"
Vídeo guardado: {{name}}

"Take PNG"
Guardar PNG

"Take Video"
Grabar Vídeo

"Ukrainian"
Ukraniano

"Wait for seconds"
Esperar durante segundos"

"Design contains tri-state I/O. You need to update your Preferences:
   Advanced features → Allow tri-state
connections
,
and reimport the block."

El diseño contiene E/S tri-estado. Debe actualizar su configuración, para ello puede hacerlo en:
Preferencias Características avanzadas → Permitir conexiones tri-estado, y volver a
importar el bloque."

Thanks for all @TimRudy ! 🚀

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.

2 participants