Skip to content

Commit

Permalink
use python version shipped with Ubuntu 22.04 (#186)
Browse files Browse the repository at this point in the history
  • Loading branch information
2bndy5 authored Feb 2, 2024
1 parent 455f328 commit bb0610e
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,8 @@ runs:
uses: actions/setup-python@v5
id: setup-python
with:
python-version: '3.11'
# use python version shipped with latest Ubuntu LTS
python-version: '3.10'
update-environment: false

- name: Install Linux clang dependencies
Expand Down

1 comment on commit bb0610e

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

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

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-format reports: 2 file(s) not formatted
  • src/demo.cpp
  • src/demo.hpp
clang-tidy reports: 7 concern(s)
  • src/demo.cpp

    C:/Program Files/Microsoft Visual Studio/2022/Enterprise/VC/Tools/MSVC/14.38.33130/include/yvals_core.h:887:1: error: [clang-diagnostic-error]

    static_assert failed "Error in C++ Standard Library usage."

    _EMIT_STL_ERROR(STL1000, "Unexpected compiler version, expected Clang 16.0.0 or newer.");
    ^
    C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\MSVC\14.38.33130\include\yvals_core.h:514:5: note: expanded from macro '_EMIT_STL_ERROR'
        static_assert(false, "Error in C++ Standard Library usage.")
        ^

    src/demo.cpp:7:8: warning: [modernize-use-trailing-return-type]

    use a trailing return type for this function

    size_t dummyFunc(size_t i) { return i; }
    ~~~~~~ ^
    auto                       -> size_t

    src/demo.cpp:9:5: warning: [modernize-use-trailing-return-type]

    use a trailing return type for this function

    int main()
    ~~~ ^
    auto       -> int

    src/demo.cpp:11:13: warning: [readability-braces-around-statements]

    statement should be inside braces

        for (;;)
                ^
                 {

    src/demo.cpp:14:5: warning: [cppcoreguidelines-pro-type-vararg]

    do not call c-style vararg functions

        printf("Hello world!\n");
        ^
  • src/demo.hpp

    src/demo.hpp:10:11: warning: [modernize-use-trailing-return-type]

    use a trailing return type for this function

        void *not_usefull(char *str){
        ~~~~~~^
        auto                         -> void *

    src/demo.hpp:12:16: warning: [modernize-use-nullptr]

    use nullptr

            return 0;
                   ^
                   nullptr

Have any feedback or feature suggestions? Share it here.

Please sign in to comment.