-
Notifications
You must be signed in to change notification settings - Fork 8
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
refactor: structure of project refactored based on standards #130
refactor: structure of project refactored based on standards #130
Conversation
@@ -5,13 +5,14 @@ import ( | |||
"os" | |||
"strings" | |||
|
|||
"github.com/pagu-project/Pagu/internal/engine" |
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.
I suggest make Pagu
lowercase here: github.com/pagu-project/pagu
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.
Need to change the repo name to github.com/pagu-project/pagu
on github and actually I have no access to do that
"github.com/pagu-project/Pagu/internal/engine" | ||
"github.com/pagu-project/Pagu/internal/engine/command" | ||
"github.com/pagu-project/Pagu/pkg/log" | ||
|
||
"github.com/pactus-project/pactus/crypto" | ||
pagu "github.com/pagu-project/Pagu" | ||
pCmd "github.com/pagu-project/Pagu/cmd" |
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.
why pCmd
?
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.
Because we have cmd *cobra.Command
as the parameter of the run function. I know this naming is not good but let's fix it in the future
Description
The project structure refactored based on this repo
Also, sub-commands have been separated into independent files
Related Issue