Skip to content

feat: support scaffolding http type packages#4116

Draft
suzuki-shunsuke wants to merge 1 commit intomainfrom
feat-support-http
Draft

feat: support scaffolding http type packages#4116
suzuki-shunsuke wants to merge 1 commit intomainfrom
feat-support-http

Conversation

@suzuki-shunsuke
Copy link
Copy Markdown
Member

No description provided.

"github.com/suzuki-shunsuke/logrus-error/logerr"
)

func convHTTPReleases(logE *logrus.Entry, cfg *Config, hrs map[string][]string) []*Release {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [golangci-lint] reported by reviewdog 🐶
calculated cyclomatic complexity for function convHTTPReleases is 11, max is 10


func (c *Controller) getPackageInfoWithVersionOverrides(ctx context.Context, logE *logrus.Entry, pkgName string, pkgInfo *registry.PackageInfo, limit int, cfg *Config) []string { //nolint:cyclop
ghReleases := c.listReleases(ctx, logE, pkgInfo, limit)
func (c *Controller) getReleases(ctx context.Context, logE *logrus.Entry, pkgInfo *registry.PackageInfo, param *config.Param, cfg *Config) ([]*Release, error) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [golangci-lint] reported by reviewdog 🐶
calculated cyclomatic complexity for function getReleases is 14, max is 10

return releases, nil
}

func (c *Controller) getPackageInfoWithVersionOverrides(ctx context.Context, logE *logrus.Entry, pkgName string, pkgInfo *registry.PackageInfo, param *config.Param, cfg *Config) []string { //nolint:cyclop
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚫 [golangci-lint] reported by reviewdog 🐶
directive //nolint:cyclop is unused for linter "cyclop"

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.

1 participant