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

[WIP] License clarification and rewrite code samples #691

Closed
wants to merge 5 commits into from

Conversation

parlough
Copy link
Contributor

@parlough parlough commented May 11, 2018

I still have to rewrite most of the code samples but I have started setting up some of the changes and clarifying licenses. People can review as I go.

In this PR I will also be updating other information as it stands in API 7.

Will address #689 when finished.

This should also close #640 and #618

@Inscrutable Inscrutable added this to the v7.X milestone May 11, 2018
@parlough
Copy link
Contributor Author

There's a long way to go, but @ST-DDT you should have push access if you ever have a chance to lend a hand. Don't really have a list to track what I've rewritten at the moment, but you can just look at files changed for now.

@parlough
Copy link
Contributor Author

Will need to modify the theme a little bit once this is done to add MIT to the footer - https://github.com/SpongePowered/sponge-docs-theme/blob/master/src/theme/templates/footer.html

@Inscrutable
Copy link
Member

There is also the possibility of using the Unlicense license for the code. http://unlicense.org/

@ST-DDT
Copy link
Member

ST-DDT commented Sep 1, 2018

Please resolve merge conflicts.

@Inscrutable
Copy link
Member

@parlough what has become of thee?

@ST-DDT
Copy link
Member

ST-DDT commented Nov 16, 2018

@parlough Can we merge most of this in, so that it does not gather any more dust? AFAIK everything can be merged except for the code samples license.

If you want I can give you a helping hand and fix the merge conflicts for you, then you just have to select the parts that can be merged earlier.

@Inscrutable
Copy link
Member

I'm afraid that @parlough has been unavailable/busy IRL for the last few months, so if you can clean this up @ST-DDT it would probably be the most expedient solution.

@ST-DDT ST-DDT mentioned this pull request Nov 18, 2018
@ST-DDT
Copy link
Member

ST-DDT commented Nov 18, 2018

Superseded by #773

@ST-DDT ST-DDT closed this Nov 18, 2018
ST-DDT added a commit that referenced this pull request Nov 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FLARD This is gathering dust
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Outdated imports in docs
3 participants