Skip to content

Cleanup #52

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 29 commits into from
Jun 1, 2024
Merged

Cleanup #52

merged 29 commits into from
Jun 1, 2024

Conversation

CalebLItalien
Copy link
Member

This closes #31 .

@CalebLItalien
Initial commit
93a1b68
@CalebLItalien
Renamed files and reorganized functions
625dc6f
@CalebLItalien
Cleanup
738ecc2
@CalebLItalien
Fixed imports in src
44f1181
@CalebLItalien
Fixed imports in testing
af8e55b
@CalebLItalien
Cleaned up old files and modularized organization
b05c55a
@CalebLItalien
Added configuration files to cargo and github
afbd2ea
@CalebLItalien
Corrected typo
16a9295
@CalebLItalien
Added name of project to workflow
1b54b97
@CalebLItalien
Resolved dependencies in src folders
852aaa0
@CalebLItalien
Fixed dependencies in tests
f8410b3
@CalebLItalien
Root documentation update
a8fb7f9
@CalebLItalien
Updated README to flow better
ab4a1a1
@CalebLItalien
Updated analysis readme
46d187d
@CalebLItalien
Update README.md
a2812f5
@CalebLItalien
Update README.md
e5e29a7
@CalebLItalien
Documented ir's src
e501d32
@CalebLItalien
Update README.md
faad952
@CalebLItalien
Updated in-code documentation
1272a3d
@CalebLItalien
Merge commit
660541b
@CalebLItalien
Moved disclaimer to README
34bd279
@CalebLItalien
Updated analysis and common README's
887e59c
@CalebLItalien
Updated logging README
5c1a2c0
@CalebLItalien
Wrote README for ir
b5f0dcd
@CalebLItalien
Added JIT README
d894d50
@CalebLItalien
Merge commit
1f74151
@CalebLItalien
Moved util to common folder
e582061
@CalebLItalien
Removed redundant documentation
b5fc296
@CalebLItalien
Fixed execute generic types
68a9a55
Dalyj24
Dalyj24 reviewed Jun 1, 2024
View reviewed changes
.github/README_TEMPLATE/README.md
Copy link
Member

Choose a reason for hiding this comment

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

Would every single function need to be documented including private functions? I understand having documentation at least for every public function

Dalyj24
Dalyj24 approved these changes Jun 1, 2024
View reviewed changes
Copy link
Member

@Dalyj24 Dalyj24 left a comment

Choose a reason for hiding this comment

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

Excellent documentation! This should be the new standard going forward

@Dalyj24 Dalyj24 merged commit a2508e2 into dev Jun 1, 2024
1 check passed
@Dalyj24 Dalyj24 deleted the cleanup branch June 1, 2024 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dalyj24 Dalyj24 Dalyj24 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@CalebLItalien @Dalyj24