openPMD / openPMD/openPMD-api

CI: C++14/17

Open
#437 0 comments 0 reactions 0 assignees View on GitHub
continuous deployment continuous integration
Dominant language
C++
Stars
161
Forks
59
Avg merge
2d 22h
Merged PRs (30d)
7

Description

Add some compile matrix entries that already build with C++14 and C++17 to see if we hit incompatibilities.

Important since e.g. conda-forge already builds with C++14 and C++17 chains.

We might also need to remove `CXX_EXTENSIONS=ON` forcing in our `CMakeLists.txt` because that should actually be controlled externally. If we remove it, we should document that we, CMake-typically, don't just build with `-std=c++11` but with the *newest found standard* such as `-std=c++1z` or `-std=gnu++17`, in case the compiler supports it.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.