Skip to content

refactor: flatten polyglot layout to standard Go module#8

Merged
Snider merged 1 commit intomainfrom
dev
Mar 21, 2026
Merged

refactor: flatten polyglot layout to standard Go module#8
Snider merged 1 commit intomainfrom
dev

Conversation

@Snider
Copy link
Copy Markdown
Contributor

@Snider Snider commented Mar 21, 2026

Move source from go/core/ to root, tests from go/tests/ to tests/. Module path dappco.re/go/core resolves cleanly — builds and tests pass.

Summary by CodeRabbit

  • Chores
    • Linting configuration has been reset. Custom settings for linter rules and issue handling have been removed, reverting the project to default linting configuration.

Move source from go/core/ to root, tests from go/tests/ to tests/.
Module path dappco.re/go/core resolves cleanly — builds and tests pass.

Co-Authored-By: Virgil <virgil@lethean.io>
@Snider Snider merged commit 3bf175b into main Mar 21, 2026
1 check was pending
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 21, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 70120759-5878-4ff0-bb7a-fadd8556bc75

📥 Commits

Reviewing files that changed from the base of the PR and between 3642a81 and fbd6464.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (48)
  • app.go
  • array.go
  • cli.go
  • command.go
  • config.go
  • contract.go
  • core.go
  • data.go
  • drive.go
  • embed.go
  • error.go
  • fs.go
  • go.mod
  • go/.golangci.yml
  • i18n.go
  • ipc.go
  • lock.go
  • log.go
  • options.go
  • runtime.go
  • service.go
  • string.go
  • task.go
  • tests/app_test.go
  • tests/array_test.go
  • tests/cli_test.go
  • tests/command_test.go
  • tests/config_test.go
  • tests/core_test.go
  • tests/data_test.go
  • tests/drive_test.go
  • tests/embed_test.go
  • tests/error_test.go
  • tests/fs_test.go
  • tests/i18n_test.go
  • tests/ipc_test.go
  • tests/lock_test.go
  • tests/log_test.go
  • tests/options_test.go
  • tests/runtime_test.go
  • tests/service_test.go
  • tests/string_test.go
  • tests/task_test.go
  • tests/testdata/cli_clir.go.bak
  • tests/testdata/scantest/sample.go
  • tests/testdata/test.txt
  • tests/utils_test.go
  • utils.go

📝 Walkthrough

Walkthrough

The entire golangci-lint configuration file was removed, clearing all linter settings, Go version specification (1.26), enabled/disabled linter lists, and issue-handling parameters.

Changes

Cohort / File(s) Summary
Configuration Removal
go/.golangci.yml
Removed all golangci-lint configuration content including run settings (timeout, Go version), enabled and disabled linter lists, and issues tuning parameters. The project will now use default golangci-lint behaviour.

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can use OpenGrep to find security vulnerabilities and bugs across 17+ programming languages.

OpenGrep is compatible with Semgrep configurations. Add an opengrep.yml or semgrep.yml configuration file to your project to enable OpenGrep analysis.

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