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

Documentation : Rework #811

Closed
wants to merge 6 commits into from

Conversation

Kreyren
Copy link
Contributor

@Kreyren Kreyren commented Jan 13, 2019

Relevant : #799

Created a concept of new documentation for README.md and CONTRIBUTION.md
My abstract is to make CONTRIBUTION.md file with full instructions
about contribution and summary of it in README.md with summary and
introduction to a repository
Added new file with contribution that should be used as full
documentation for contributors, will also add images to make reading
more effective.

Looking for feedback

Kreyren and others added 6 commits January 9, 2019 13:10
Bug : PhoenicisOrg#792
Changes WINE environment from winxp to win7 -> winxp outputs error about unsupported OS.
Changes executable to workaround PhoenicisOrg#792
Tested, working
Signed-off-by: Jacob Hrbek <werifgx@gmail.com>
Relevant : PhoenicisOrg#799

Created a concept of new documentation for README.md and CONTRIBUTION.md
My abstract is to make `CONTRIBUTION.md` file with full instructions
about contribution and summary of it in README.md with summary and
introduction to a repository
Added new file with contribution that should be used as full
documentation for contributors, will also add images to make reading
more effective.
Looking for feedback
Signed-off-by: Jacob Hrbek <werifgx@gmail.com>
This is a concept for new README.md, still work in progress
Signed-off-by: Jacob Hrbek <werifgx@gmail.com>
@qparis
Copy link
Member

qparis commented Jan 13, 2019

Seems that you have removed Hearthstone files. We cannot merge

@Kreyren
Copy link
Contributor Author

Kreyren commented Jan 13, 2019

@qparis i think i've removed them just from tracking, checking..

@Kreyren
Copy link
Contributor Author

Kreyren commented Jan 13, 2019

Based on informations from git on freenode it's correct, but i'm going to recreate it just to be sane.

@Kreyren Kreyren closed this Jan 13, 2019
@Kreyren
Copy link
Contributor Author

Kreyren commented Jan 13, 2019

Requesting removal of #811, reason spam.

Kreyren pushed a commit to Kreyrenicis/compatibilators that referenced this pull request Mar 7, 2019
Removed vcrun2008, vcrun2017, corefonts, adobeair and dxd9 since dependencies for league of legends are stagend.
Added hotfix for PhoenicisOrg#881

Signed-off-by: Jacob Hrbek <werifGX@gmail.com>
@Kreyren Kreyren mentioned this pull request Mar 8, 2019
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.

None yet

2 participants