omissis/goarkitect

Similar blocks of code found in 2 locations. Consider refactoring.

Open

func (e haveContentMatchingRegex) getViolation(filePath string) rule.CoreViolation {
    format := "file '%s' does not have content matching regex '%s'"

    if e.options.matchSingleLines {
        format = "file '%s' does not have all lines matching regex '%s'"

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

func (e haveContentMatchingExpression) getViolation(filePath string) rule.CoreViolation {
    format := "file '%s' does not have content matching '%s'"

    if e.options.matchSingleLines {
        format = "file '%s' does not have all lines matching '%s'"

• Read up
  Read up

Method RuleBuilder.Because has 7 return statements (exceeds 4 allowed).

Open

func (rb *RuleBuilder) Because(b rule.Because) ([]rule.Violation, []error) {
    if rb.locked {
        rb.addLockError()

        return nil, rb.GetErrors()

Function ExecuteFileRule has 5 return statements (exceeds 4 allowed).

Open

func ExecuteFileRule(conf Rule) ([]rule.Violation, []error) {
    rb, err := createFileBuilder(conf)
    if err != nil {
        return nil, []error{err}
    }

Method App.Run has 5 return statements (exceeds 4 allowed).

Open

func (a *App) Run() error {
    if err := a.flagSet.Parse(GetArgs(os.Args, 1)); err != nil {
        return fmt.Errorf("error parsing flags: %w", err)
    }