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

feat: Maintainability and much more... #8

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

carlocorradini
Copy link

Improving maintainability 🥳🤗🤯
The CHANGELOG.md file contains every relevant changes.
The modifications are listed in the list below:

  • All includes must be preceded by the word erkir: #include <erkir/FILE.h>
  • Single include header file, erkir.h, exports all library's component
  • Tests implemented with GoogleTest
  • CI enhancements (you can push a tag and a new release is automatically created only if all checks pass)
  • Overall code and repository structure have been reworked to improve maintainability

@vahancho I was also considering changing the extension of all header files from .h to .hpp because this is a "pure" C++ project rather than a C one (and also good practice 😊).

Please let me know what you think 🥳🤗🤯

@KrukDallan
Copy link

Awesome! The includes are now more useful, thanks!😊

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.

2 participants