-
-
Notifications
You must be signed in to change notification settings - Fork 21
refactor: The whole project #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
b309799
6d3251d
6f4c655
d81576c
c53985b
ae49475
38c9661
731daa5
b2b216c
24a8a91
1731bab
bcdc939
d758750
2c7426f
d58a38c
291b496
4e8aa39
f47aba9
5c2d709
1886a0f
e93dd0a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,2 +1,5 @@ | ||
| result/ | ||
| hexecute | ||
| # go build -o bin ./... | ||
| bin/ | ||
|
|
||
| # nix build (produces a symlink) | ||
| result | ||
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,214 @@ | ||||||||
| package main | ||||||||
|
|
||||||||
| import ( | ||||||||
| "encoding/json" | ||||||||
| "flag" | ||||||||
| "log" | ||||||||
| "os" | ||||||||
| "runtime" | ||||||||
| "time" | ||||||||
|
|
||||||||
| "github.com/ThatOtherAndrew/Hexecute/internal/config" | ||||||||
| "github.com/ThatOtherAndrew/Hexecute/internal/draw" | ||||||||
| "github.com/ThatOtherAndrew/Hexecute/internal/execute" | ||||||||
| gestures "github.com/ThatOtherAndrew/Hexecute/internal/gesture" | ||||||||
| "github.com/ThatOtherAndrew/Hexecute/internal/models" | ||||||||
| "github.com/ThatOtherAndrew/Hexecute/internal/opengl" | ||||||||
| "github.com/ThatOtherAndrew/Hexecute/internal/spawn" | ||||||||
| "github.com/ThatOtherAndrew/Hexecute/internal/stroke" | ||||||||
| "github.com/ThatOtherAndrew/Hexecute/internal/update" | ||||||||
| "github.com/ThatOtherAndrew/Hexecute/pkg/wayland" | ||||||||
| "github.com/go-gl/gl/v4.1-core/gl" | ||||||||
| ) | ||||||||
|
|
||||||||
| func init() { | ||||||||
| runtime.LockOSThread() | ||||||||
| } | ||||||||
|
|
||||||||
| type App struct { | ||||||||
| *models.App | ||||||||
| } | ||||||||
|
|
||||||||
|
Comment on lines
+28
to
+31
|
||||||||
| type App struct { | |
| *models.App | |
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finally, accurate information from AI! The issue has been resolved in this PR, though it currently doesn’t affect the project.
Copilot
AI
Oct 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] The local variable gestures shadows the imported package name 'gestures', which reduces readability and can lead to subtle mistakes. Consider renaming the variable to 'gs' or 'loadedGestures'.
Copilot
AI
Oct 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] The local variable spawn shadows the imported package 'spawn'. Rename the variable (e.g., 'spawner') to avoid confusion; the same pattern appears again when spawning cursor sparkles.
Copilot
AI
Oct 16, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] Same shadowing as above; rename the local variable to avoid masking the package import.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| module hexecute | ||
| module github.com/ThatOtherAndrew/Hexecute | ||
|
|
||
| go 1.25.1 | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| package config | ||
|
|
||
| import ( | ||
| "os" | ||
| "path/filepath" | ||
| ) | ||
|
|
||
| func GetPath() (string, error) { | ||
| homeDir, err := os.UserHomeDir() | ||
| if err != nil { | ||
| return "", err | ||
| } | ||
| configDir := filepath.Join(homeDir, ".config", "hexecute") | ||
| if err := os.MkdirAll(configDir, 0755); err != nil { | ||
| return "", err | ||
| } | ||
| return filepath.Join(configDir, "gestures.json"), nil | ||
| } |
Uh oh!
There was an error while loading. Please reload this page.