Skip to content

Chore/upgrade go version#205

Merged
cybersiddhu merged 7 commits into
developfrom
chore/upgrade-go-version
May 16, 2026
Merged

Chore/upgrade go version#205
cybersiddhu merged 7 commits into
developfrom
chore/upgrade-go-version

Conversation

@cybersiddhu
Copy link
Copy Markdown
Member

No description provided.

Upgrade go.mod, Dockerfile, and all CI workflow files to use Go 1.25.
Docker image updated to golang:1.25-trixie (bullseye not available for 1.25).

💘 Generated with Crush

Assisted-by: Crush:deepseek/deepseek-v4-pro
Create a documentation file to centralize common CLI commands for testing,
linting, and formatting. This provides a quick reference for developers
to ensure consistent execution of project tasks using gotestsum and
golangci-lint.
…hecks

Update the golangci-lint configuration to version 2, enable parallel
runners, and refresh the list of enabled linters to align with modern
Go best practices. Add specific formatters and refine exclusion rules
to reduce noise from generated code and third-party dependencies.
- golang.org/x/{oauth2,sync,text,net,crypto,sys,exp} to latest
- google.golang.org/{api,grpc,protobuf,genproto} to latest
- Transitive deps (cloud.google.com, otel) updated accordingly

💘 Generated with Crush

Assisted-by: Crush:deepseek/deepseek-v4-pro
- Replace magic numbers with named constants across all files
- Fix testifylint expected-actual ordering and assert.Contains
- Modernize interface{}→any and SplitSeq usage
- Fix line length, whitespace, and formatting (wsl_v5, gofumpt, golines)
- Replace deprecated option.WithCredentialsFile with WithAuthCredentialsFile

💘 Generated with Crush

Assisted-by: Crush:deepseek/deepseek-v4-pro
Replace the default github.token with a custom REPO_ACCESS_TOKEN secret.
The default token often lacks the necessary permissions to trigger
subsequent workflows or perform specific repository actions, requiring a
personal access token with broader scope to ensure the pull request
creation process succeeds.
Update the linting workflow to utilize the centralized golang-lint
reusable workflow from the dictyBase/workflows repository. This change
also restricts the workflow to ignore the master branch and ensures
the linter runs against the specific pull request head commit.
@codecov
Copy link
Copy Markdown

codecov Bot commented May 16, 2026

Codecov Report

❌ Patch coverage is 1.85185% with 53 lines in your changes missing coverage. Please review.
✅ Project coverage is 12.58%. Comparing base (5dea5ce) to head (13e53ff).

Files with missing lines Patch % Lines
internal/app/dagger/dagger.go 0.00% 7 Missing ⚠️
internal/app/issue/issue.go 0.00% 7 Missing ⚠️
internal/app/chart/chart.go 0.00% 5 Missing ⚠️
internal/app/analytics/analytics.go 0.00% 4 Missing ⚠️
internal/app/comment/ontology.go 20.00% 4 Missing ⚠️
internal/app/repository/commit.go 0.00% 4 Missing ⚠️
internal/app/repository/file.go 0.00% 4 Missing ⚠️
internal/file/io.go 0.00% 4 Missing ⚠️
internal/github/manager.go 0.00% 4 Missing ⚠️
internal/app/deploy/deploy.go 0.00% 2 Missing ⚠️
... and 4 more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #205      +/-   ##
===========================================
- Coverage    18.92%   12.58%   -6.35%     
===========================================
  Files           38       38              
  Lines         2029     1812     -217     
===========================================
- Hits           384      228     -156     
+ Misses        1604     1558      -46     
+ Partials        41       26      -15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cybersiddhu cybersiddhu merged commit bacc7f9 into develop May 16, 2026
3 of 5 checks passed
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