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

Wrap all sources in a namespace #20

Open
fosterbrereton opened this issue Sep 22, 2021 · 2 comments
Open

Wrap all sources in a namespace #20

fosterbrereton opened this issue Sep 22, 2021 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@fosterbrereton
Copy link
Contributor

To eat our own dogfood here, the sources should be wrapped in a namespace to minimize the chances of ODRVs happen. For source-level structures and algorithms, they should be wrapped in an anonymous namespace (I believe most - but maybe not all - already are.)

@fosterbrereton fosterbrereton added the bug Something isn't working label Sep 22, 2021
@kc9jud
Copy link

kc9jud commented Sep 23, 2021

Somehow it feels appropriate that this project should have a CI config that builds the source, and then runs the built artifact on its own source.

Maybe if I get some time I'll come up with a PR.

@fosterbrereton
Copy link
Contributor Author

Eating our own dogfood here sounds amazing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants