Skip to content

Commit

Permalink
refactor(cmd): mv src/cmd/config.zig to src/config.zig (#582)
Browse files Browse the repository at this point in the history
I find it confusing that we have a cmd.zig in the sources root, and then
we have a separate cmd folder that has nothing except the application
configuration. I don't see why this folder exists if it doesn't contain
cmd.zig. Either the folder should contain both cmd.zig and config.zig,
or the folder should be eliminated and both go in src. Dividing it in
half is just confusing and seems like the worst approach. In this PR
I've opted to eliminate the folder.
  • Loading branch information
dnut authored Feb 27, 2025
1 parent 0c7b01b commit 1048b6a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/cmd.zig
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ const builtin = @import("builtin");
const cli = @import("zig-cli");
const sig = @import("sig.zig");
const std = @import("std");
const config = @import("cmd/config.zig");
const config = @import("config.zig");

const AccountsDB = sig.accounts_db.AccountsDB;
const FullAndIncrementalManifest = sig.accounts_db.FullAndIncrementalManifest;
Expand Down
2 changes: 1 addition & 1 deletion src/cmd/config.zig → src/config.zig
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
const std = @import("std");
const sig = @import("../sig.zig");
const sig = @import("sig.zig");

const ShredNetworkConfig = sig.shred_network.ShredNetworkConfig;
const resolveSocketAddr = sig.net.net.resolveSocketAddr;
Expand Down

0 comments on commit 1048b6a

Please sign in to comment.