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

Renamed files using convention verb-object.client... #850

Merged
merged 1 commit into from
Aug 27, 2015
Merged

Renamed files using convention verb-object.client... #850

merged 1 commit into from
Aug 27, 2015

Conversation

vaucouleur
Copy link
Contributor

Renamed files using file naming convention: verb-object.client...

Closes #817

@rhutchison
Copy link
Contributor

LGTM

1 similar comment
@mleanos
Copy link
Member

mleanos commented Aug 25, 2015

LGTM

@lirantal
Copy link
Member

@vaucouleur this PR has conflicts now after we merged the facebook PR and #759
can you take a look?

@vaucouleur
Copy link
Contributor Author

@lirantal I just rebased using upstream. Can you please try again.

@rhutchison
Copy link
Contributor

Thanks @vaucouleur LGTM

@mleanos
Copy link
Member

mleanos commented Aug 25, 2015

LGTM as well

@codydaig
Copy link
Member

LGTM

@rhutchison rhutchison mentioned this pull request Aug 26, 2015
15 tasks
Renamed files using convention verb-object.client.etc. Closes #817
@vaucouleur
Copy link
Contributor Author

@lirantal Rebased using upstream following #774

@codydaig
Copy link
Member

Still LGTM

Thanks @vaucouleur for keeping up on this PR with all the changes and rebases.

@mleanos
Copy link
Member

mleanos commented Aug 26, 2015

Tested. LGTM.

ilanbiala added a commit that referenced this pull request Aug 27, 2015
Rename files using convention verb-object.client
@ilanbiala ilanbiala merged commit a004b66 into meanjs:master Aug 27, 2015
@ilanbiala
Copy link
Member

@lirantal just fyi I merged so @rhutchison could continue working on something that needs this.

@lirantal lirantal added this to the 0.4.x milestone Aug 27, 2015
@lirantal
Copy link
Member

Of course, no problems.

@ilanbiala ilanbiala modified the milestones: 0.4.1, 0.4.x Aug 31, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants