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

Fix goimports. #984

Closed
bwplotka opened this issue Mar 27, 2019 · 13 comments
Closed

Fix goimports. #984

bwplotka opened this issue Mar 27, 2019 · 13 comments

Comments

@bwplotka
Copy link
Member

AC:

  • make format or gomiports should fix blank lines.

Eg:

 	"net/http"
 	"path"
 	"regexp"
 	"sort"
 
+	thanosrule "github.com/improbable-eng/thanos/pkg/rule"
+
 	"github.com/improbable-eng/thanos/pkg/store/storepb"
 
 	"github.com/go-kit/kit/log"
 	"github.com/prometheus/client_golang/prometheus"
 	"github.com/prometheus/common/route"

goimports does NOT fixes those, see why: golang/go#26246

This is quite annoying. I think this was fixed, but then reverted. Anyway we should ensure this is fixed.

@stale
Copy link

stale bot commented Jan 28, 2020

This issue/PR is has been automatically marked as stale because it has not had recent activity. Please comment on status otherwise the issue will be closed in a week. Thank you for your contributions.

@stale stale bot added the stale label Jan 28, 2020
@jojohappy
Copy link
Member

I saw it had been fixed with pinned goimports. Any update? @bwplotka

@stale stale bot removed the stale label Jan 28, 2020
@bwplotka
Copy link
Member Author

Still valid.

@stale
Copy link

stale bot commented Feb 27, 2020

This issue/PR has been automatically marked as stale because it has not had recent activity. Please comment on status otherwise the issue will be closed in a week. Thank you for your contributions.

@stale stale bot added the stale label Feb 27, 2020
@stale stale bot closed this as completed Mar 5, 2020
@bwplotka
Copy link
Member Author

bwplotka commented Mar 5, 2020

Same problem.

@bwplotka bwplotka reopened this Mar 5, 2020
@stale stale bot removed the stale label Mar 5, 2020
@stale
Copy link

stale bot commented Apr 4, 2020

This issue/PR has been automatically marked as stale because it has not had recent activity. Please comment on status otherwise the issue will be closed in a week. Thank you for your contributions.

@stale stale bot added the stale label Apr 4, 2020
@bwplotka
Copy link
Member Author

bwplotka commented Apr 5, 2020 via email

@stale stale bot removed the stale label Apr 5, 2020
@stale
Copy link

stale bot commented May 5, 2020

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label May 5, 2020
@bwplotka
Copy link
Member Author

bwplotka commented May 5, 2020 via email

@stale stale bot removed the stale label May 5, 2020
@stale
Copy link

stale bot commented Jun 4, 2020

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jun 4, 2020
@bwplotka
Copy link
Member Author

bwplotka commented Jun 5, 2020 via email

@stale stale bot removed the stale label Jun 5, 2020
@stale
Copy link

stale bot commented Jul 5, 2020

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jul 5, 2020
@stale
Copy link

stale bot commented Jul 12, 2020

Closing for now as promised, let us know if you need this to be reopened! 🤗

@stale stale bot closed this as completed Jul 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants