-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add missing includes with the help of inshpect #1129
Merged
Merged
Changes from 61 commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
621ce9d
Add inshpect CI action
msimberg dfab9f8
Simplify inshpect config file
msimberg 7e7144e
Enable inshpect includes check
msimberg 83ea397
Add more file extensions to inshpect config
msimberg 9b3f002
Fix missing <memory> and <set> includes
msimberg a26be9c
Fix missing <unordered_map> includes
msimberg 986080c
Add missing <string> includes
msimberg 8ad52be
add mc.h file with the new include
fe17144
add 80 #include <utility>
0501bfe
add #include <utility>, in 70 files
4031cf2
add #include <sstream>, in 10 files, #include <array>
f39982b
aa #include <functional> in 9 files
faabe04
add last #include <utility>
4a856c3
add #include <type_traits> in 4 files
481135b
add #include <vector> in all files
bef420b
last add, vector, utily, ecc.
d1999a9
refactor code for solve check error
3dac06d
Refactor of the code
f900113
add inshpect.yml
e03c713
add inshpect.yml file
8b1387a
add install tools
f0b014a
add other two steps: Checkout, Inshpect
0ca4519
Reformat of the code and merge with origin/master
7151621
add install insphect
86f20f0
add set -x
f053415
change the url for dasel with the latest version
cd817d0
change url for insphect
d6d367d
change url for inshpect download
3d7b483
add ripgrep download directly
5146bce
debug
336e9d6
FINISH
d5a1f62
Update .github/workflows/inshpect.yml
f1371aa
Remove from my stage inshpect.output
6cb82b0
Review changes
b98fb7e
Update impl.h
gulivarese 650dd85
Update impl.h
gulivarese 6e1929f
Update impl.h
gulivarese 17d14da
Update kernel_runner.h
gulivarese a130366
Update utils.h
gulivarese a09b80e
Update invoke.h
gulivarese 9db3874
Update invoke.h
gulivarese 9ab061e
Remove set -x in inshpect.yml
9d7bf60
Refresh with curl
5a37d4f
add two pattern about #include <typeinfo>
7e4a43f
modify settings of VScode
0c22ea3
Add #include <typeinfo>
bdb33a6
Update inshpect.yml, callable_object.h, invoke.h
d88ad21
Update .github/workflows/inshpect.yml
af4993d
Update inshpect.yml file
d804bbc
Refactor code
a318873
Restore from origin/master about check_format.yml
8eae802
Move #include <utility>
633f8eb
Delete two white lines and add one lines before curl command
bb01f90
add echo and sha256sum --check for dasel
ed6e5a9
Add pushd and popd for dasel
00e9edf
Remove sudo command for dasel
575f44b
remove path for dasel
b21b982
Move dependencies for inshpect before this
cc17993
Debug: add set -x
990b000
Add d for dasel
704d41f
Remove set -x for debug
77dce97
Add newline in print_csv.h
e1696fb
Install ripgrep and dasel with apt command, delete the formal install…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,46 @@ | ||
name: Inshpect | ||
|
||
on: | ||
pull_request: | ||
|
||
jobs: | ||
inshpect: | ||
runs-on: ubuntu-22.04 | ||
|
||
steps: | ||
- name: Install tools | ||
run: | | ||
sudo apt update | ||
sudo apt install -y fd-find | ||
|
||
pushd /tmp | ||
curl --output dasel \ | ||
--location https://github.com/TomWright/dasel/releases/download/v2.7.0/dasel_linux_amd64 | ||
echo "38decc82e0dcf9a5ccd22918b909a78c92ee0d919b4534b0a6d90c802936b1a0 dasel" | sha256sum --check | ||
chmod +x dasel | ||
mv dasel /usr/local/bin | ||
popd | ||
|
||
pushd /tmp | ||
curl --output ripgrep.tar.gz \ | ||
--location https://github.com/BurntSushi/ripgrep/releases/download/13.0.0/ripgrep-13.0.0-x86_64-unknown-linux-musl.tar.gz | ||
echo "ee4e0751ab108b6da4f47c52da187d5177dc371f0f512a7caaec5434e711c091 ripgrep.tar.gz" | sha256sum --check | ||
tar --extract --strip-components 1 --file ripgrep.tar.gz | ||
mv rg /usr/local/bin | ||
popd | ||
|
||
pushd /tmp | ||
curl --output inshpect \ | ||
--location https://raw.githubusercontent.com/msimberg/inshpect/d31ba8ed7e636c57cb9085ed3a8450d998e6b8a5/inshpect | ||
echo "4b891a178fa99ac800b9b92acc0596d72f640ca1ae6de7f35ac4b3785dcd7af9 inshpect" | sha256sum --check | ||
chmod +x inshpect | ||
mv inshpect /usr/local/bin/inshpect | ||
popd | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
- name: Inshpect | ||
if: always() | ||
run: | | ||
export INSHPECT_NUMTHREADS=4 | ||
export INSHPECT_FD=fdfind | ||
inshpect . .inshpect.toml |
RMeli marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,176 @@ | ||
# | ||
# Distributed Linear Algebra with Future (DLAF) | ||
# | ||
# Copyright (c) 2018-2023, ETH Zurich | ||
# All rights reserved. | ||
# | ||
# Please, refer to the LICENSE file in the root directory. | ||
# SPDX-License-Identifier: BSD-3-Clause | ||
# | ||
|
||
extensions = ["c", "cpp", "cu", "h", "h.in"] | ||
|
||
[includes] | ||
enable = true | ||
patterns = [ | ||
{ pattern = '\bstd::make_shared\b', include = 'memory' }, | ||
{ pattern = '\bstd::map\b', include = 'map' }, | ||
{ pattern = '\bstd::set\b', include = 'set' }, | ||
{ pattern = '\bstd::multi_map\b', include = 'multi_map' }, | ||
{ pattern = '\bstd::multi_set\b', include = 'multi_set' }, | ||
{ pattern = '\bstd::unordered_map\b', include = 'unordered_map' }, | ||
{ pattern = '\bstd::unordered_set\b', include = 'unordered_set' }, | ||
{ pattern = '\bstd::unordered_multi_map\b', include = 'unordered_map' }, | ||
{ pattern = '\bstd::unordered_multi_set\b', include = 'unordered_set' }, | ||
{ pattern = '\bstd::((shared|unique)_ptr)\b', include = 'memory' }, | ||
{ pattern = '\bstd::addressof\b', include = 'memory' }, | ||
{ pattern = '\bstd::list\b', include = 'list' }, | ||
{ pattern = '\bstd::string\b', include = 'string' }, | ||
{ pattern = '\bstd::vector\b', include = 'vector' }, | ||
{ pattern = '\bstd::array\b', include = 'array' }, | ||
{ pattern = '\bstd::reference_wrapper\b', include = 'functional' }, | ||
{ pattern = '\bstd::(c?ref)\b', include = 'functional' }, | ||
{ pattern = '\bstd::(u?int[0-9]+_t)\b', include = 'cstdint' }, | ||
{ pattern = '\bstd::((i|o)?stringstream)\b', include = 'sstream' }, | ||
{ pattern = '\bstd::((i|o)?fstream)\b', include = 'fstream' }, | ||
{ pattern = '\bstd::(cin|cout|cerr|clog)\b', include = 'iostream' }, | ||
{ pattern = '\bstd::setw\b', include = 'iomanip' }, | ||
{ pattern = '\bstd::setprecision\b', include = 'iomanip' }, | ||
{ pattern = '\bstd::size_t\b', include = 'cstddef' }, | ||
{ pattern = '\bstd::ptrdiff_t\b', include = 'cstddef' }, | ||
{ pattern = '\bstd::nullptr_t\b', include = 'cstddef' }, | ||
{ pattern = '\bstd::(true|false)_type\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::(bool|integral)_constant\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::invoke_result\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::invoke_result_t\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_void\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_void_t\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_null_pointer\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_integral\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_floating_point\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_array\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_enum\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_union\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_class\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_function\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_pointer\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_lvalue_reference\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_rvalue_reference\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_member_object_pointer\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_member_function_pointer\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_fundamental\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_arithmetic\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_scalar\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_object\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_compound\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_reference\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_member_pointer\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_const\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_volatile\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivial\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_copyable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_standard_layout\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_pod\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_literal_type\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_empty\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_polymorphic\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_abstract\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_signed\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_unsigned\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_nothrow_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_default_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_default_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_nothrow_default_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_copy_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_copy_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_nothrow_copy_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_move_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_move_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_nothrow_move_constructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_nothrow_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_copy_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_copy_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_nothrow_copy_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_move_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_move_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_nothrow_move_assignable\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_destructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_trivially_destructible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_same\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_base_of\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_convertible\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::has_virtual_destructor\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::alignment_of\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::aligned_storage\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::aligned_union\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::rank\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::extent\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::add_(cv|const|pointer|volatile|lvalue_reference|rvalue_reference)\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::make_signed\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::make_unsigned\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::remove_(cv|const|volatile|reference|pointer|extent|all_extents)\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::decay\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::enable_if\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::conditional\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::common_type\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::underlying_type\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::result_of\b', include = 'type_traits' }, | ||
{ pattern = '\bstd::is_invocable([a-z_]*)\b', include = 'type_traits' }, | ||
{ pattern = '\btypeid\b', include = 'typeinfo' }, | ||
{ pattern = '\bstd::type_info\b', include = 'typeinfo' }, | ||
{ pattern = '\bstd::(mem((set)|(cpy)|(move)))\b', include = 'cstring' }, | ||
{ pattern = '\bstd::swap\b', include = 'utility' }, | ||
{ pattern = '\bstd::move\b', include = 'utility' }, | ||
{ pattern = '\bstd::forward\b', include = 'utility' }, | ||
{ pattern = '\bstd::declval\b', include = 'utility' }, | ||
{ pattern = '\bstd::pair\b', include = 'utility' }, | ||
{ pattern = '\bstd::make_pair\b', include = 'utility' }, | ||
{ pattern = '\bstd::in_place\b', include = 'utility' }, | ||
{ pattern = '\bstd::exception_ptr\b', include = 'exception' }, | ||
{ pattern = '\bstd::terminate\b', include = 'exception' }, | ||
{ pattern = '\bstd::(s|g)et_terminate\b', include = 'exception' }, | ||
{ pattern = '\bstd::swap_ranges\b', include = 'algorithm' }, | ||
{ pattern = '\bstd::iter_swap\b', include = 'algorithm' }, | ||
{ pattern = '\bstd::atomic\b', include = 'atomic' }, | ||
{ pattern = '\bstd::(memory_order_[a-z_]+)\b', include = 'atomic' }, | ||
{ pattern = '\bstd::(minstd_rand(0)?)\b', include = 'random' }, | ||
{ pattern = '\bstd::(mt19937(_64)?)\b', include = 'random' }, | ||
{ pattern = '\bstd::random_device\b', include = 'random' }, | ||
{ pattern = '\bstd::uniform_(int|uniform)_distribution\b', include = 'random' }, | ||
{ pattern = '\bstd::apply\b', include = 'tuple' }, | ||
{ pattern = '\bstd::(forward_as_|make_|)tuple\b', include = 'tuple' }, | ||
{ pattern = '\bstd::ignore\b', include = 'tuple' }, | ||
{ pattern = '\bstd::tie\b', include = 'tuple' }, | ||
{ pattern = '\bstd::tuple_(cat|element|size)\b', include = 'tuple' }, | ||
{ pattern = '\bstd::(make_)optional\b', include = 'optional' }, | ||
{ pattern = '\bstd::nullopt\b', include = 'optional' }, | ||
{ pattern = '\bstd::any(|_cast)\b', include = 'any' }, | ||
{ pattern = '\bstd::error_(code|condition|category)\b', include = 'system_error' }, | ||
{ pattern = '\bstd::system_error\b', include = 'system_error' }, | ||
{ pattern = '\bstd::chrono\b', include = 'chrono' }, | ||
] | ||
|
||
[copyright] | ||
enable = false | ||
|
||
[license] | ||
enable = false | ||
|
||
[spdx] | ||
enable = false | ||
|
||
[pragma_once] | ||
enable = false | ||
|
||
[deprecated_includes] | ||
enable = false | ||
|
||
[deprecated_names] | ||
enable = false | ||
|
||
[disallowed_macros] | ||
enable = false |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,6 +12,7 @@ | |
|
||
/// @file | ||
|
||
#include <optional> | ||
#include <utility> | ||
|
||
#include <pika/async_rw_mutex.hpp> | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,7 @@ | |
|
||
/// @file | ||
|
||
#include <cstddef> | ||
#include <vector> | ||
|
||
#include <dlaf/common/assert.h> | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,6 +13,7 @@ | |
/// @file | ||
|
||
#include <type_traits> | ||
#include <utility> | ||
|
||
#include <pika/execution.hpp> | ||
|
||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,7 @@ | |
#pragma once | ||
|
||
#include <complex> | ||
#include <functional> | ||
#include <utility> | ||
|
||
#include <mpi.h> | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,7 @@ | |
#pragma once | ||
|
||
#include <complex> | ||
#include <functional> | ||
#include <utility> | ||
|
||
#include <mpi.h> | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -13,6 +13,7 @@ | |
/// @file | ||
|
||
#include <complex> | ||
#include <functional> | ||
#include <utility> | ||
|
||
#include <mpi.h> | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,7 @@ | |
#pragma once | ||
|
||
#include <complex> | ||
#include <functional> | ||
#include <utility> | ||
|
||
#include <mpi.h> | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be nice to run it already on ubuntu 24.04
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, I add this review!
Thanks @rasolca !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a very good point @rasolca. @gulivarese this means that you can now install
dasel
andripgrep
directly with apt.dasel
was not yet available in 22.04, andripgrep
was not compiled with PCRE2 in 22.04, so they both had to be installed manually with 22.04. With 24.04 both of these are fixed.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@msimberg so now I need to
runs-on: ubuntu-22.04
and then I can delete the two steps below about the installation of the two dependencies (ripgrep and dasel).Is it correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need
runs-on: ubuntu-24.04
(note 24, not 22). You of course need to install them with apt instead and since I haven't tried it myself the ubuntu versions may not be compatible, but give it a try and then we'll see if it works.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, sorry I made an mistake when I wrote
22.04
I pushed my modify and my updates.