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

quick clean up #2

Merged
merged 2 commits into from
Mar 21, 2017
Merged

quick clean up #2

merged 2 commits into from
Mar 21, 2017

Conversation

wzrdtales
Copy link

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

Copy link
Owner

@dgonzalez dgonzalez left a 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.

"private": true,
"main": "econ-refused.js",
"author": "",
"license": "ISC",
Copy link
Owner

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)

Copy link
Owner

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.

Copy link
Author

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

@wzrdtales
Copy link
Author

@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?

@dgonzalez
Copy link
Owner

No, repo will be transferred to my account. This is the right one. Thanks for your changes!

@dgonzalez dgonzalez merged commit ba18bac into dgonzalez:master Mar 21, 2017
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.

2 participants