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

SwagDoc para Delphi XE6 #21

Open
wos83 opened this issue May 26, 2023 · 5 comments
Open

SwagDoc para Delphi XE6 #21

wos83 opened this issue May 26, 2023 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@wos83
Copy link

wos83 commented May 26, 2023

Marcelo, tudo bom?
Parabens pelo trabalho com SwagDoc.
Fiz uma adaptação para o Delphi XE6, subi no meu repositório https://github.com/wos83/swagdoc-delphi-xe6/ fique a vontede para atualizar no seu e a galera poder usar em mais versões.
Muito Grato e Viva ao Delphi, risos!

@marcelojaloto
Copy link
Owner

Olá,
Tudo bem e você?

Fico contente que gostou da biblioteca. Muito obrigado pelos agradecimentos.

Poderia me passar maiores detalhes e necessidades sobre suas alterações no SwagDoc?

Vou analisar seu código também e tão logo já faço o merge das suas alterações.

Depois aviso o DMVC para sincronizar as alterações também, porque o SwagDoc foi incorporado no projeto DMVC.

@wos83
Copy link
Author

wos83 commented May 29, 2023

Fala Marcelo, então estou programando para um projeto num cliente aonde ele tem somente o Delphi XE6..
O que mudei foi colocar algumas diretivas de compilação e adaptar para aceitar a bibliotecas do XE6.
Fico grato em ajudar.

@marcelojaloto marcelojaloto added the enhancement New feature or request label Jun 1, 2023
@marcelojaloto
Copy link
Owner

@wos83 bom dia! Analisei suas alterações e farei a sincronização! Por favor, poderia fazer a gentileza de gerar um pull request e enviar para eu aprovar e fazer o merge?

@wos83
Copy link
Author

wos83 commented Jun 15, 2023

Combinado, acabei de fazer. 🚀

@marcelojaloto
Copy link
Owner

@wos83 olá fiz as revisões e queria saber se estar de acordo com duas sugestões que fiz no pull request. São devidas a compatibilidade com outras versões do Delphi.
Ok? Você poderia ajustar e me notificar no PR?
Assim ficamos com a mesma versão.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants