Skip to content

Aplicando Popcorngp-detekt-rule#95

Merged
gabrielbmoro merged 2 commits into
masterfrom
gmoro/popcorngp-detekt-rule
Jun 24, 2026
Merged

Aplicando Popcorngp-detekt-rule#95
gabrielbmoro merged 2 commits into
masterfrom
gmoro/popcorngp-detekt-rule

Conversation

@gabrielbmoro

Copy link
Copy Markdown
Collaborator

Descrição

Aplicando popcorngp-detekrule no projeto

Checklist

  • Os testes foram executados e passaram com sucesso.
  • As alterações de código seguem as diretrizes de estilo do projeto.
  • Foram adicionados testes, se aplicável.
  • Se inscreveu no canal?😛

@gabrielbmoro gabrielbmoro requested a review from a team as a code owner June 7, 2026 16:57
"packagePrefix": "com.codandotv.streamplayerapp",
"rules": [
{
"filePattern": "^.*/data/.+\\.kt$",

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

nenhuma classe de data pode depender de domain e presentation

]
},
{
"filePattern": "^.*/domain/.+\\.kt$",

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

nenhuma classe de domain pode depender de presentation

]
},
{
"filePattern": "^.*/.+Handler\\.kt$",

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

classes que possuem a palavra handler não devem ter dependências internas

@gabrielbmoro gabrielbmoro merged commit 527120d into master Jun 24, 2026
4 checks passed
@gabrielbmoro gabrielbmoro deleted the gmoro/popcorngp-detekt-rule branch June 24, 2026 18:27
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