Agrupar problemas levantados pelo Codacy e gerar issues do github

[teles]

A tela de issues do Codacy já agrupa as issues.

Vou levantar quantos tipos tem de issues hoje e priorizar quais devem ser resolvidos primeiro de acordo com o custo benefício da solução.

[alisonmonteiro]

@teles estou trabalhando para resolver um erros da categoria Error Prone, no Codacy. O qual, pede para definir a função antes da utilização da mesma.

Pensei em duas formas de otimizar isto, por exemplo:

var app = angular.module('frontpress.apis.posts');

function postsApi() { 
  // ...
}

app.factory('PostsApi', postsApi);

angular
  .module('frontpress.apis.posts')
  .factory('PostsApi', function() {
    // ...
  });

Já utilizei as duas, e não tive problemas. Qual você acha mais interessante? Tem outra ideia?

Vlw! 😄

[teles]

Opa, @alisonmonteiro! Muito obrigado pela ajuda e pelas sugestões. Gostei mais da primeira versão por ser mais explícita.

Acho que ao invés de:

var app = angular.module('frontpress.apis.posts');

poderia ser

var postsApiModule = angular.module('frontpress.apis.posts');

mas entendo que isso tornaria inviável uma alteração em massa, então acho que poderíamos mudar tudo para app ou module de início e depois ir mudando aos poucos para deixar o nome da variável mais explícita.

O que acha?

[alisonmonteiro]

Opa, @teles! :)

Para comerçar poderiamos usar a sugestão abaixo e aos poucos ir melhorando.

var module = angular.module() // ...

Outra coisa que gostaria de ver o que você acha. Na criação de módulos como este, você prefere qual abordagem?

var module = angular.module('frontpress', [ 
  'frontpress.views', 
  'frontpress.apis.blog', 
  'frontpress.components.slugs-map',
  'frontpress.components.frontpress-provider', 
  'frontpress.template-cache'
]);

var module = angular.module('frontpress', ['frontpress.views', 'frontpress.apis.blog', 'frontpress.components.slugs-map','frontpress.components.frontpress-provider', 'frontpress.template-cache']);

Eu, particularmente, prefiro a primeira. Acho que dá uma leitura bem melhor. O que você acha?

[teles]

A primeira abordagem realmente é bem melhor :) @alisonmonteiro