Skip to content

Develop - #24

Merged
Benevanio merged 2 commits into
masterfrom
develop
Mar 24, 2026
Merged

Benevanio merged 2 commits into
masterfrom
develop

Conversation

@Benevanio

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 24, 2026 20:43
@Benevanio
Benevanio merged commit eb577ac into master Mar 24, 2026
4 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the README’s Electron desktop app section to better support distribution/testing of the Windows build.

Changes:

  • Adds a “Para testadores” callout describing what’s needed to run the desktop app.
  • Adds step-by-step instructions for test users to install and run the app via the NSIS installer.
  • Updates the installer output note to reference the expected .exe path/name in dist-electron/.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Comment on lines +119 to +121
> **Para testadores:** o executavel `.exe` e o unico arquivo necessario para usar o app. Nao e preciso instalar Node.js, Docker ou qualquer dependencia adicional — somente ter o **Google Chrome** ou **Microsoft Edge** instalado na maquina.

O projeto pode ser executado como app desktop Windows, sem precisar subir Docker no computador do usuario final.

Copilot AI Mar 24, 2026

Copy link

Choose a reason for hiding this comment

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

O texto sugere que o .exe e o unico arquivo necessario para usar o app, mas este .exe parece ser o instalador (NSIS) e o app roda a partir do que foi instalado (atalho/arquivos). Para evitar interpretacao de "app portatil", sugiro ajustar para algo como "o instalador .exe e o unico arquivo necessario para instalar e usar" e/ou explicitar que o requisito de Chrome/Edge e para executar o scraper (o app pode abrir sem ele, mas a busca falhara).

Suggested change
> **Para testadores:** o executavel `.exe` e o unico arquivo necessario para usar o app. Nao e preciso instalar Node.js, Docker ou qualquer dependencia adicional — somente ter o **Google Chrome** ou **Microsoft Edge** instalado na maquina.
O projeto pode ser executado como app desktop Windows, sem precisar subir Docker no computador do usuario final.
> **Para testadores:** o instalador `.exe` e o unico arquivo necessario para **instalar e usar** o app. Nao e preciso instalar Node.js, Docker ou qualquer dependencia adicional — apenas ter o **Google Chrome** ou **Microsoft Edge** instalado para que o scraper (busca de vagas) funcione corretamente.
Apos a instalacao, o projeto pode ser executado como app desktop Windows, sem precisar subir Docker no computador do usuario final.

Copilot uses AI. Check for mistakes.
Comment thread README.md
```

O instalador sera gerado em `dist-electron/` (target `nsis`).
O instalador sera gerado em `dist-electron/Vagas Full Setup X.X.X.exe`.

Copilot AI Mar 24, 2026

Copy link

Choose a reason for hiding this comment

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

O nome do arquivo gerado pelo electron-builder geralmente inclui a versao do package.json (ex.: Vagas Full Setup 1.0.0.exe) e pode vir acompanhado de outros artefatos no dist-electron/. Para manter a doc precisa, considere dizer "sera gerado em dist-electron/ (ex.: Vagas Full Setup <versao>.exe)" em vez de afirmar um caminho/nome exato.

Suggested change
O instalador sera gerado em `dist-electron/Vagas Full Setup X.X.X.exe`.
O instalador sera gerado em `dist-electron/` (ex.: `Vagas Full Setup <versao>.exe`).

Copilot uses AI. Check for mistakes.
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