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

Disable gofmt for non go1.2 or go1.3 #322

Merged
merged 1 commit into from
Jul 1, 2014

Conversation

brendandburns
Copy link
Contributor

No description provided.

@lavalamp
Copy link
Member

lavalamp commented Jul 1, 2014

LGTM

@proppy proppy mentioned this pull request Jul 1, 2014
@lavalamp
Copy link
Member

lavalamp commented Jul 1, 2014

Travis failed on go1.2 and the error message isn't helpful...

@proppy
Copy link
Contributor

proppy commented Jul 1, 2014

Yes, previous version was printing the name of the file that were not formatted.


bad=$(gofmt -s -l pkg/ cmd/)
if [ "$bad" != "0" ]; then
exit 1
Copy link
Contributor

Choose a reason for hiding this comment

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

consider echo'ing $bad

proppy added a commit to proppy/kubernetes that referenced this pull request Jul 1, 2014
workaround gofmt failure, until kubernetes#322 is in
@lavalamp
Copy link
Member

lavalamp commented Jul 1, 2014

@proppy turned off building at tip, so rebase, turn tip back on, and fix this at your leisure. :)

@brendandburns
Copy link
Contributor Author

Fixed, ptal.

lavalamp added a commit that referenced this pull request Jul 1, 2014
Disable gofmt for non go1.2 or go1.3
@lavalamp lavalamp merged commit cf2cf50 into kubernetes:master Jul 1, 2014
vishh pushed a commit to vishh/kubernetes that referenced this pull request Apr 6, 2016
seans3 pushed a commit to seans3/kubernetes that referenced this pull request Apr 10, 2019
wking pushed a commit to wking/kubernetes that referenced this pull request Jul 21, 2020
implement Resources func for applicationImpl
benluddy added a commit to benluddy/kubernetes that referenced this pull request Sep 30, 2022
linxiulei pushed a commit to linxiulei/kubernetes that referenced this pull request Jan 18, 2024
Bump base image debian-base to tag v1.0.0 to pick up some CVE fixes
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.

3 participants