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

Author name must be between 5 and 50 characters #261

Open
ruzz opened this issue Mar 4, 2014 · 3 comments
Open

Author name must be between 5 and 50 characters #261

ruzz opened this issue Mar 4, 2014 · 3 comments

Comments

@ruzz
Copy link

ruzz commented Mar 4, 2014

I'm trying to release my first package and i'm having a bit of an issue. It makes it through adding the tag and pushing the repo but fails on the author name. you can see my author name in the smart.json file is well over 5 characters.

could this possibly be my 4 letter github name causing an issue?

$ mrt release .
Adding tag and pushing repo
Error: Package could not be saved
Author name must be between 5 and 50 characters

$ mrt publish .
Error: Package could not be saved
Author name must be between 5 and 50 characters

$ cat smart.json
{
"name": "reactive-variable",
"description": "a slightly more flexible way to do reactive variables",
"homepage": "https://github.com/ruzz/Meteor-ReactiveVariable.git",
"author": "i.m. ruzz ([email protected])",
"version": "0.3.3",
"git": "https://github.com/ruzz/Meteor-ReactiveVariable.git"
}

$ whoami
ruzz

any thoughts?

@yeputons
Copy link
Contributor

yeputons commented Mar 4, 2014

@ruzz Looks more like Atmosphere's issue. It's because parseAuthor function allows [A-Za-z0-9_] and spaces only in name - no dots. Therefore, your name is parsed as i. One more comment: email is supposed to be inside <>, () is for URLs.

@tmeasday What do you think about fixing it? May be we should create a separate issue in Atmosphere's repo?

@ruzz
Copy link
Author

ruzz commented Mar 4, 2014

yeah, i had just arrived at the parsing problem myself. I'll make a few quick changes and let you know what happens. and thanks for the formatting tip on the email.

@ruzz
Copy link
Author

ruzz commented Mar 4, 2014

hey hey. look at that. works now. thanks ;)

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

No branches or pull requests

2 participants