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

Jump to bottom

testing: document that example output must use // (and avoid gofmt flakiness) #4812

Closed
gopherbot opened this issue Feb 15, 2013 · 5 comments
Closed

testing: document that example output must use // (and avoid gofmt flakiness) #4812

gopherbot opened this issue Feb 15, 2013 · 5 comments
Labels
FrozenDueToAge
Milestone
Go1.1

Comments

@gopherbot
Copy link

by daviddengcn:

Before filing a bug, please check whether it has been fixed since the
latest release. Search the issue tracker and check that you're running the
latest version of Go:

Run "go version" and compare against
http://golang.org/doc/devel/release.html  If a newer version of Go exists,
install it and retry what you did to reproduce the problem.

Thanks.

What steps will reproduce the problem?
If possible, include a link to a program on play.golang.org.
1.
2.
3.

What is the expected output?
After "go fmt" my example cases are not failed

What do you see instead?
They are failed by tabs

Which compiler are you using (5g, 6g, 8g, gccgo)?

Which operating system are you using?
MacOS


Which version are you using?  (run 'go version')
1.0.3

Please provide any additional information below.
More information: https://groups.google.com/forum/#!topic/golang-nuts/ZH5YgwmjCXk
@rsc
Copy link
Contributor

rsc commented Feb 15, 2013

Comment 1:

Let's document that example output must use // comments.

@natefinch
Copy link
Contributor

Comment 2:

Is there any interest in just fixing it so that a common indent is stripped off?  An
example here:
http://play.golang.org/p/u5P7NZ_Cvj

@rsc
Copy link
Contributor

rsc commented Feb 15, 2013

Comment 3:

I would rather just document it. Stripping common indent is full of
problems too.

@davecheney
Copy link
Contributor

Comment 4:

Issue #4811 has been merged into this issue.

@adg
Copy link
Contributor

adg commented Feb 22, 2013

Comment 6:

This issue was closed by revision 2482ef7.

Status changed to Fixed.

@rsc rsc added this to the Go1.1 milestone Apr 14, 2015
@rsc rsc removed the go1.1maybe label Apr 14, 2015
@golang golang locked and limited conversation to collaborators Jun 24, 2016
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Go1.1
Development

No branches or pull requests
5 participants
@davecheney @rsc @natefinch @adg @gopherbot