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 pt1 #593

Merged
merged 37 commits into from
May 4, 2019
Merged

Documentation pt1 #593

merged 37 commits into from
May 4, 2019

Conversation

Wyqer
Copy link
Member

@Wyqer Wyqer commented Apr 23, 2019

Q A
Bug fix? no
New feature? no
Needs wipe? no
Fixed issues #508

Description:

Various documentation work on the new framework.

Content:

  • Documentation for the general structure of the framework
  • Module information via readme and Wiki/Website
    • Init
    • Common
    • Core
    • Persistance
    • Respawn
    • Admin
    • Resources
    • Permission
    • Arsenal
    • Player menu
    • Virtual
    • Build
    • Garrison
    • Logistic
    • Enemy
    • Cratefiller
  • Functions library (for public functions)
    • Public function attribute for every function
    • Optional: Examples for each function
  • Instructions how to add a module from scratch via the provided example module
  • Our basic coding conventions/contribution guidelines for the public
  • Replacement of the diag_log function with our own logging function KPLIB_fnc_common_log
  • More stuff which will be added here

@Wyqer Wyqer added this to the Sprint 12 milestone Apr 23, 2019
@Wyqer
Copy link
Member Author

Wyqer commented Apr 23, 2019

@KillahPotatoes/kplib-testers
As we've renamed some functions/modules in the commits, a short run on this branch would be good. Just to be sure we haven't messed up or missed to change something due to the renaming.
Thanks in advance.

@Wyqer Wyqer marked this pull request as ready for review May 4, 2019 06:42
@Wyqer Wyqer merged commit b7ce1df into v0.97S12 May 4, 2019
@Wyqer Wyqer deleted the v0.97S12-508 branch May 4, 2019 06:47
@Wyqer Wyqer added done and removed in progress labels May 5, 2019
@Wyqer Wyqer mentioned this pull request May 5, 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.

1 participant