-
Notifications
You must be signed in to change notification settings - Fork 2
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: move pasta shared para dentro da pasta modules #9
Conversation
Guia do Revisor por SourceryEste pull request move o arquivo 'font1.ttf.import' do diretório 'Shared/Resources' para o diretório 'Modules/Shared/Resources'. Nenhum diagrama gerado, pois as alterações parecem simples e não necessitam de uma representação visual. Alterações em Nível de Arquivo
Dicas e comandosInteragindo com o Sourcery
Personalizando sua ExperiênciaAcesse seu painel para:
Obtendo Ajuda
Original review guide in EnglishReviewer's Guide by SourceryThis pull request moves the 'font1.ttf.import' file from the 'Shared/Resources' directory to the 'Modules/Shared/Resources' directory. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Olá @patrickreisx - Revisei suas alterações - aqui estão alguns comentários:
Comentários Gerais:
- Por favor, use inglês para títulos e descrições de PRs para manter a consistência na comunicação do projeto. Considere renomear isso para 'refactor: move shared folder into modules directory'
Aqui está o que eu analisei durante a revisão
- 🟢 Questões Gerais: tudo parece bom
- 🟢 Segurança: tudo parece bom
- 🟢 Testes: tudo parece bom
- 🟢 Complexidade: tudo parece bom
- 🟢 Documentação: tudo parece bom
Sourcery é gratuito para código aberto - se você gostou de nossas revisões, considere compartilhá-las ✨
Original comment in English
Hey @patrickreisx - I've reviewed your changes - here's some feedback:
Overall Comments:
- Please use English for PR titles and descriptions to maintain consistency in project communication. Consider renaming this to 'refactor: move shared folder into modules directory'
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
LGTM! |
Resumo por Sourcery
Tarefas de manutenção:
Original summary in English
Summary by Sourcery
Chores: