-
Notifications
You must be signed in to change notification settings - Fork 14
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
quick clean up #2
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Just change the license and I will merge it. This is the correct repo.
examples/econ-refused/package.json
Outdated
"private": true, | ||
"main": "econ-refused.js", | ||
"author": "", | ||
"license": "ISC", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
License should be MIT (as per base repo)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Applies for all the other examples.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure will Update this just quickly Cherry picked the old One
be65519
to
794c49d
Compare
794c49d
to
1e0a6cb
Compare
@dgonzalez Pushed the requested changes out. About the repo constellation. It is quite a struggle to open a PR in this constellation as you always end up by default in rjdoger ones. Is there a special reason for this structure? |
No, repo will be transferred to my account. This is the right one. Thanks for your changes! |
As talked the transfer from dgonzalez/visigoth-old#2
not sure where it should be opened up on, its there in both places now:
rjrodger#3