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

chore: Added new e-sport module to be implemented (#768) #787

Conversation

Mehdibenhadjkhelifa
Copy link
Contributor

@Mehdibenhadjkhelifa Mehdibenhadjkhelifa commented Jul 4, 2024

Added the e-sport module boilerplate code as requested. Although i don't know if the implementations were part of the request . Also changed cmake under /src to include both .cpp and .h files. If more work regarding this task is needed just let me know. Cheers !
Edit: Checks failed, probably because i forgot to export the functions and include them in the /include directory. I'm working on it.

Copy link
Owner

@cieslarmichal cieslarmichal left a comment

Choose a reason for hiding this comment

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

Yup, implementation should be included with tests, also add doxygen comments in header

@cieslarmichal cieslarmichal linked an issue Jul 5, 2024 that may be closed by this pull request
Copy link
Owner

@cieslarmichal cieslarmichal left a comment

Choose a reason for hiding this comment

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

use Esport/esport not E-sport/e-sport

@cieslarmichal
Copy link
Owner

hey are you working on this or should I close a PR?

@Mehdibenhadjkhelifa
Copy link
Contributor Author

Hey, I'm not currently working on it no. if someone else wants to take the job let them have it.

@cieslarmichal
Copy link
Owner

Ok

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.

create e-sport module
2 participants