Skip to content
This repository was archived by the owner on Sep 18, 2024. It is now read-only.

Desbord#69

Closed
lesdieuxx wants to merge 40 commits intodashboardfrom
master
Closed

Desbord#69
lesdieuxx wants to merge 40 commits intodashboardfrom
master

Conversation

@lesdieuxx
Copy link

Please describe the changes this PR makes and why it should be merged, as well as any and all proof of successful tests:

Semantic versioning classification:

  • This PR changes the framework's interface (methods or parameters added)
    • This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
  • This PR only includes non-code changes, like changes to README, etc.

nathanielcwm and others added 30 commits November 19, 2017 18:55
minor improvements namely for the times when the help msg is too long
Bumped minor and patch version up by 1
Fixed `awaitReply`, was assigning not comparing author ID's
Updated license to 2018 and current license holder.
…#43)

* Add client.getGuildSettings method, adjusted all files to use it. Update to djs 11.3.2

* More tweaks for using overrides for settings instead of all-or-nothing

* Revert to using NPM version of enmap, muh bad

* Tweak help because I'm a derpy derp.
Fixed a typo in the client.unloadCommand() error message.
Simply delete require.cache[module] results in a memory leak after some time as the deleted module is still referenced in require.cache[module].parent.children, preventing it from GCing. The new function will allow it to GC properly.
MIT License

Copyright (c) 2017 Évelyne Lachance
Copyright (c) 2018 YorkAARGH
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should remain as Évelyne Lachance as this version of GuideBot is her code.

@YorkAARGH
Copy link
Member

No changes after 8 days at the time of writing this message, conflicting files are unresolved, and the copyright claim hasn't been fixed either.

@YorkAARGH YorkAARGH closed this Nov 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.