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

added CMakeFiles !!! #20

Closed
wants to merge 1 commit into from

Conversation

amantham20
Copy link
Member

@amantham20 amantham20 commented Sep 27, 2023

  • Created an overlapping Cmakefile to run the project

giphy

- Updated to include the tests
@amantham20
Copy link
Member Author

amantham20 commented Sep 28, 2023

changes made:

  • added ignore for all of the build folders.
  • Created an overlapping Cmakefiles that manages both tests and source code.
  • made a sample run file called sample

things to potentially change in the future.

  • making get catch2 part of cmakelists
  • moving executable for source files into a cmakelists in source files

@FergusonAJ
Copy link
Contributor

This PR looks good, though I wanted to clean up a couple things before merging.

Since I don't have write perms to https://github.com/amantham20/CSE491_GP_Group7/tree/makefile-2-cmake , I instead built off this PR and made a new one at #27

@amantham20 amantham20 added the Depricated no more needed label Oct 5, 2023
FergusonAJ pushed a commit that referenced this pull request Dec 6, 2023
Fixed formatting of entity file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants