Skip to content

[v10] Reformat imports GCI #18746

Merged
jakule merged 6 commits intobranch/v10from
jakule/v10/reformat-gci
Dec 16, 2022
Merged

[v10] Reformat imports GCI #18746
jakule merged 6 commits intobranch/v10from
jakule/v10/reformat-gci

Conversation

@jakule
Copy link
Copy Markdown
Contributor

@jakule jakule commented Nov 23, 2022

Backport of #17894
I backported the GCI changes and reformat all files to avoid resolving all conflicts.

TODO:

  • Merge teleport.e PR

@github-actions github-actions Bot added application-access audit-log Issues related to Teleports Audit Log backport bpf Used to bugs with bpf and enhanced session recording. database-access Database access related issues and PRs desktop-access kubernetes-access machine-id rdp tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Nov 23, 2022
Copy link
Copy Markdown
Contributor

@codingllama codingllama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for backporting.

Let's get the e/ ref updated and a green CI run, otherwise happy to approve.

Comment thread build.assets/Dockerfile Outdated
@jakule jakule force-pushed the jakule/v10/reformat-gci branch from 220738d to 6d4578e Compare November 30, 2022 22:23
@jakule jakule force-pushed the jakule/v10/reformat-gci branch from 6d4578e to 8e74d1a Compare December 16, 2022 19:39
@jakule jakule requested a review from codingllama December 16, 2022 19:41
Comment thread build.assets/Dockerfile
@github-actions github-actions Bot removed the request for review from r0mant December 16, 2022 19:44
@codingllama
Copy link
Copy Markdown
Contributor

Thanks for the backport!

@jakule jakule enabled auto-merge (squash) December 16, 2022 19:52
@jakule jakule merged commit 6ee8d32 into branch/v10 Dec 16, 2022
fheinecke pushed a commit that referenced this pull request Dec 16, 2022
* Add GCI linter and format target

* Revert golangci-lint to 1.46.0

* Disable GCI in golangci-lint

* Downgrade GCI to 0.7.0

* GCI all files

* Suppress GCI/goimports linter errors
@zmb3 zmb3 deleted the jakule/v10/reformat-gci branch May 7, 2024 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

application-access audit-log Issues related to Teleports Audit Log backport bpf Used to bugs with bpf and enhanced session recording. database-access Database access related issues and PRs desktop-access kubernetes-access machine-id rdp tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants