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

App name validation fix #451

Merged
merged 1 commit into from
Feb 29, 2016

Conversation

digitalfishpond
Copy link
Contributor

#411

App name is restricted to 24 chars and regex pattern now also restricts first char to lowercase letter

Review on Reviewable

@codecov-io
Copy link

Current coverage is 81.76%

Merging #451 into master will not affect coverage as of 4da32f0

@@            master    #451   diff @@
======================================
  Files           77      77       
  Stmts          625     625       
  Branches         0       0       
  Methods          0       0       
======================================
  Hit            511     511       
  Partial          0       0       
  Missed         114     114       

Review entire Coverage Diff as of 4da32f0

Powered by Codecov. Updated on successful CI builds.


/**
* Maximum length for Application name
* @export {string}
*/
this.nameMaxLength = '63';
this.nameMaxLength = '24';
Copy link
Contributor

Choose a reason for hiding this comment

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

test must be adapted as well

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cheld PTAL

@digitalfishpond digitalfishpond force-pushed the service-name-maxchars branch 2 times, most recently from 127ad06 to f1510f5 Compare February 26, 2016 12:29
@cheld
Copy link
Contributor

cheld commented Feb 26, 2016

LGTM

@bryk
Copy link
Contributor

bryk commented Feb 29, 2016

:lgtm:


Reviewed 2 of 3 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

bryk added a commit that referenced this pull request Feb 29, 2016
@bryk bryk merged commit 8ab33d9 into kubernetes:master Feb 29, 2016
@digitalfishpond digitalfishpond deleted the service-name-maxchars branch March 2, 2016 08:40
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.

5 participants