Skip to content

Conversation

@robert-andrzejuk
Copy link
Collaborator

Very simple merge of README file from first SCOPE project

Very simple merge of README file from first SCOPE project
JeffGarland and others added 19 commits February 4, 2025 18:05
- remove the identity boilerplate
- drop in the un-merged readme content from original scope repo
- indicate header only library
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@JeffGarland JeffGarland changed the title Update README.md Update README.md - and attempt to satisfy ci and remove exemplar cruft Feb 5, 2025
Copy link
Member

@JeffGarland JeffGarland left a comment

Choose a reason for hiding this comment

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

I'm approving this review. @robert-andrzejuk anytime you're ready to merge to main I'm good. This looks like a good enough line to snap as a starting point

@robert-andrzejuk
Copy link
Collaborator Author

I'm approving this review. @robert-andrzejuk anytime you're ready to merge to main I'm good. This looks like a good enough line to snap as a starting point

Are we going to worry about the lint checks, clang-format checks, etc??

@JeffGarland
Copy link
Member

The checks are a little over the top in my view. I started a thread about it in discourse bc most of these things like whitespace in markdown I think don't matter for any flavor of markdown? In which case, why should we check that? The code formatting is another matter -- we'll want to fix that at some point, but I care more about getting something compiling for the moment.

@robert-andrzejuk robert-andrzejuk merged commit 6ecb758 into main Feb 5, 2025
4 of 128 checks passed
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.

4 participants