Skip to content
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

Order import section by config #90

Merged
merged 5 commits into from
Aug 1, 2022
Merged

Order import section by config #90

merged 5 commits into from
Aug 1, 2022

Conversation

vorotislav
Copy link
Contributor

@vorotislav vorotislav commented Jul 27, 2022

Closes #74

added the ability to change the order of processing of import sections by config

@vorotislav vorotislav changed the title Orser import section by config Order import section by config Jul 27, 2022
@daixiang0
Copy link
Owner

Please add a flag for this feature since some users do not want to config order.

Signed-off-by: Tetiuev Pavel <[email protected]>
Signed-off-by: vorotislav <[email protected]>
pkg/gci/internal/testdata/order.cfg.yaml Outdated Show resolved Hide resolved
pkg/gci/internal/testdata/order2.cfg.yaml Outdated Show resolved Hide resolved
cmd/gci/gcicommand.go Outdated Show resolved Hide resolved
cmd/gci/gcicommand.go Outdated Show resolved Hide resolved
Signed-off-by: vorotislav <[email protected]>

rename tests
README.md Outdated Show resolved Hide resolved
pkg/config/config.go Outdated Show resolved Hide resolved
pkg/gci/gci.go Outdated Show resolved Hide resolved
pkg/gci/gci.go Outdated Show resolved Hide resolved
Signed-off-by: vorotislav <[email protected]>
pkg/gci/gci.go Outdated Show resolved Hide resolved
Signed-off-by: vorotislav <[email protected]>
@daixiang0 daixiang0 merged commit c32c832 into daixiang0:master Aug 1, 2022
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.

support config the order of sections
3 participants