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

Feature/add missing packageproject options #524

Merged
merged 4 commits into from
Jan 17, 2024
Merged

Feature/add missing packageproject options #524

merged 4 commits into from
Jan 17, 2024

Conversation

ClausKlein
Copy link
Contributor

Fix typos with codespell too

TheLartians
TheLartians requested changes Dec 7, 2023
View reviewed changes
Copy link
Member

@TheLartians TheLartians left a comment

Choose a reason for hiding this comment

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

Thanks a lot for updating the options and the spell-check! Just a small request before merging this.

test/style/CMakeLists.txt Outdated Show resolved Hide resolved
ClausKlein and others added 2 commits December 7, 2023 17:45
@ClausKlein @TheLartians
Update test/style/CMakeLists.txt
0c75493 
Co-authored-by: Lars Melchior <TheLartians@users.noreply.github.com>
@ClausKlein
fix-cmake-format
b53d0c3
@ClausKlein
Copy link
Contributor Author

Thanks a lot for updating the options and the spell-check! Just a small request before merging this.

It was right formated?

TheLartians
TheLartians approved these changes Jan 17, 2024
View reviewed changes
Copy link
Member

@TheLartians TheLartians left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@TheLartians TheLartians merged commit 3c25130 into cpm-cmake:master Jan 17, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TheLartians TheLartians TheLartians 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
@ClausKlein @TheLartians