Skip to content

Conversation

@erikh
Copy link
Contributor

@erikh erikh commented Feb 28, 2017

This fixes numerous gometalinter (make lint) related issues by applying fixes and (sometimes large) necessary refactors.

This is a WIP and will be finished as I get time, which probably means this week. There is plenty to review already, however, if you are so inclined. Most new changes will end up near the end of the commit list.

PTAL

Fixes #5

@erikh
Copy link
Contributor Author

erikh commented Feb 28, 2017

this also incorporates some other branches, very sorry. this is in no shape for merge yet though.

@erikh erikh force-pushed the fix-lint branch 2 times, most recently from 66146f9 to 7fc10e9 Compare March 4, 2017 10:20
@erikh
Copy link
Contributor Author

erikh commented Mar 4, 2017

This is ready for review now I think. Note that it currently incorporates #26
and #24 so things will pass.

Erik Hollensbe added 9 commits March 6, 2017 11:14
…ired

This rewrites btrfs_noversion to btrfs_version and
libdm_no_deferred_remove to libdm_deferred_remove. This allows many
systems to compile without build tags; the current situation is that
most hosts require the build tags and very few do not.

Signed-off-by: Erik Hollensbe <github@hollensbe.org>
Signed-off-by: Erik Hollensbe <github@hollensbe.org>
Signed-off-by: Erik Hollensbe <github@hollensbe.org>
Signed-off-by: Erik Hollensbe <github@hollensbe.org>
Signed-off-by: Erik Hollensbe <github@hollensbe.org>
Signed-off-by: Erik Hollensbe <github@hollensbe.org>
Signed-off-by: Erik Hollensbe <github@hollensbe.org>
Signed-off-by: Erik Hollensbe <github@hollensbe.org>
Signed-off-by: Erik Hollensbe <github@hollensbe.org>
@erikh
Copy link
Contributor Author

erikh commented Mar 6, 2017

rebased over #26

@rhatdan
Copy link
Member

rhatdan commented Mar 27, 2017

Needs a rebase.

@rhatdan
Copy link
Member

rhatdan commented Apr 19, 2017

@erikh Can you fix up this pull request?

@erikh
Copy link
Contributor Author

erikh commented Apr 19, 2017 via email

@rhatdan rhatdan closed this Jun 23, 2017
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.

Fix lint issues

2 participants