Skip to content
This repository has been archived by the owner on Aug 28, 2021. It is now read-only.

remove no-longer-needed extraneous subdirs #3687

Closed
wants to merge 2 commits into from
Closed

remove no-longer-needed extraneous subdirs #3687

wants to merge 2 commits into from

Conversation

wardn
Copy link
Contributor

@wardn wardn commented Sep 8, 2017

No description provided.

@arv
Copy link
Contributor

arv commented Sep 9, 2017

Thanks for the PR.

Can this wait? I have a huge patch in progress which touches a lot of files. It would be better to do the reorg when we do not have large patches in flight.

@cmasone-attic
Copy link
Contributor

cmasone-attic commented Sep 10, 2017 via email

@aboodman
Copy link
Contributor

@cmasone-attic yes there was, it me: #3668.

Thanks @wardn for the patch. I'm in favor of this. The useless /go subdir has been bugging me for awhile.

@aboodman
Copy link
Contributor

PS @wardn in order to accept any patch from you we'll need you to complete the contributor agreement first: https://github.com/attic-labs/noms/blob/master/CONTRIBUTING.md#contributing-code

@wardn
Copy link
Contributor Author

wardn commented Sep 10, 2017

fixed merge conflicts

@wardn wardn closed this Sep 14, 2017
@wardn wardn deleted the jsremove branch September 14, 2017 17:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants