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.

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

Documentation and clean up #15

Merged
merged 12 commits into from
Apr 23, 2024
Merged

Documentation and clean up #15

merged 12 commits into from
Apr 23, 2024

Conversation

Thoemi09
Copy link
Contributor

@Thoemi09 Thoemi09 commented Apr 5, 2024

This PR is very similar to the one from itertools. The following things have been done:

  • formatted .clang-tidy file
  • added c++ documentation
  • split mpi.hpp into multiple header files
  • removed all sphinx and added doxygen related files
  • updated the github action workflow

@Thoemi09 Thoemi09 requested a review from Wentzell April 5, 2024 21:04
@Thoemi09 Thoemi09 force-pushed the DEV_DOCUMENTATION branch from f7213a2 to 6b3f5e7 Compare April 11, 2024 14:31
@Wentzell Wentzell force-pushed the DEV_DOCUMENTATION branch from 6b3f5e7 to 8d1e3aa Compare April 23, 2024 16:19
Wentzell
Wentzell approved these changes Apr 23, 2024
View reviewed changes
Copy link
Member

@Wentzell Wentzell left a comment

Choose a reason for hiding this comment

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

Thank you @Thoemi09 for this huge improvement! Only made some minor changes, this all looks good now.

@Wentzell Wentzell merged commit 9823241 into TRIQS:unstable Apr 23, 2024
2 of 4 checks passed
@Thoemi09 Thoemi09 deleted the DEV_DOCUMENTATION branch April 23, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Wentzell Wentzell Wentzell 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
@Thoemi09 @Wentzell