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

Pull Request - Adiconar altrerações #2

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

Conversation

Rtrevisan20
Copy link

Português
Nesta Pull as seguintes alterações e inclusões foram feitas:

  • Melhorado arquivo README.md em breve adicionarei um README. em ingles
  • Renomeado o projeto TurboUpdate para TurboUpdate.Model
  • Criado um novo projeto chamado TurboUpdateCore
  • Adicionado uma forma de traduzir as consts, usando uma inteface e um arquivo .inc
  • Reposicionei os projetos FMX e VCL em uma pasta própria
  • Adicionado uma function Killtask na unit TurboUpdate.Model.Utils para ser usada quando em stadalone
  • Adicionado uma forma de mostrar as mensagem usando o HDMesageDlg
  • Adiconado versionamento semâtico para usar o BOSS
  • Adicionado unit TurboUpdate.Model.Update.Thread para evitar o acoplamento

Inglês
In this Pull the following changes and additions were made:

  • Improved README.md file soon I will add a README. in English
  • Renamed the TurboUpdate project to TurboUpdate.Model
  • Created a new project called TurboUpdateCore
  • Added a way to translate consts, using an interface and a .inc file
  • I repositioned the FMX and VCL projects in their own folder
  • Added a Killtask function in the TurboUpdate.Model.Utils unit to be used when in stadalone
  • Added a way to display messages using HDMesageDlg
  • Added semantic versioning to use BOSS
  • Added TurboUpdate.Model.Update.Thread unit to avoid coupling

@errorcalc
Copy link
Owner

Hi,
Since your changes change the API for the user, I'll take some time to create a separate "legacy branch" for the old API and review all the changes for master branch.
I think within a month your changes will be accepted.

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.

None yet

2 participants