Skip to content

feat!:(2.0 global services)#156

Merged
jacoobes merged 151 commits intomainfrom
global_services
Dec 28, 2022
Merged

feat!:(2.0 global services)#156
jacoobes merged 151 commits intomainfrom
global_services

Conversation

@jacoobes
Copy link
Copy Markdown
Member

@jacoobes jacoobes commented Nov 3, 2022

sern 2.0!!!

  • Test it at your own risk, this PR is in active development. If you decide to test and find some bugs, make sure to report it in our server

INCLUDES BREAKING CHANGES!

…t_of_module-manager

# Conflicts:
#	package-lock.json
#	package.json
#	src/handler/events/observableHandling.ts
#	src/types/handler.ts
@jacoobes jacoobes marked this pull request as ready for review December 15, 2022 16:36
# Conflicts:
#	README.md
#	package-lock.json
#	package.json
#	src/handler/sern.ts
#	src/handler/structures/context.ts
#	src/handler/structures/wrapper.ts
#	src/handler/utilities/treeSearch.ts
#	src/types/handler.ts
@EvolutionX-10
Copy link
Copy Markdown
Member

review is in progress, will do it some files at a time 😅

Copy link
Copy Markdown
Member

@EvolutionX-10 EvolutionX-10 left a comment

Choose a reason for hiding this comment

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

why test/esm/src/index.ts is empty file?

Comment thread .eslintrc
Comment thread src/handler/events/messageHandler.ts
Comment thread src/handler/structures/enums.ts
Comment thread src/handler/utilities/functions.ts
@jacoobes
Copy link
Copy Markdown
Member Author

Was gonna add testing but I'm lazy

Copy link
Copy Markdown
Member

@Murtatrxx Murtatrxx 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 overall, but needs small improvements

Comment thread src/handler/dependencies/provider.ts
Comment thread src/handler/structures/errors.ts Outdated
@jacoobes jacoobes merged commit 1455622 into main Dec 28, 2022
@jacoobes jacoobes deleted the global_services branch December 28, 2022 20:24
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.

4 participants