godexsoft
038b2d2f35
[CI] clang-tidy auto fixes
2025-05-07 09:37:45 +00:00
Ayaz Salikhov
68f832a832
ci: Use XRPLF/clio-dev-team in dependabot ( #2081 )
2025-05-07 09:41:59 +01:00
Ayaz Salikhov
5e43c3b099
test: Remove SetUp and TearDown methods in tests ( #2086 )
...
Fix: https://github.com/XRPLF/clio/issues/910
2025-05-07 09:41:23 +01:00
Ayaz Salikhov
9af36e5235
ci: run pre-commit workflow on heavy runner to make doxygen work ( #2085 )
2025-05-06 15:51:00 +01:00
Ayaz Salikhov
90103431ea
fix: Pass secrets.CODECOV_TOKEN explicitly ( #2079 )
2025-05-02 17:17:27 +01:00
Ayaz Salikhov
ed27c41641
ci: Fail CI if codecov upload fails ( #2078 )
2025-05-02 16:17:32 +01:00
Alex Kremer
3daa735b31
fix: Flaky test in ETLng ( #2077 )
...
Fixes a flaky test
Fix: https://github.com/XRPLF/clio/issues/2075
---------
Co-authored-by: Ayaz Salikhov <mathbunnyru@users.noreply.github.com >
2025-05-02 15:55:12 +01:00
Ayaz Salikhov
5d2c2a8bfc
ci: Move clang build analyzer to nightly ( #2074 )
2025-05-02 13:50:02 +01:00
Ayaz Salikhov
f2384a47f1
feat: Implement clang build analyzer ( #2072 )
...
Fix: https://github.com/XRPLF/clio/issues/936
2025-05-02 12:49:51 +01:00
Ayaz Salikhov
c774a40a59
ci: Install ClangBuildAnalyzer in the ci image ( #2071 )
2025-05-01 18:49:57 +01:00
Ayaz Salikhov
477d8e1f5f
ci: Don't use concurrency with workflow_call ( #2069 )
2025-05-01 18:03:46 +01:00
Ayaz Salikhov
7fd51d8a60
ci: Refactor how we run code coverage ( #2065 )
...
Fix: https://github.com/XRPLF/clio/issues/2061
2025-05-01 16:44:37 +01:00
Ayaz Salikhov
b2a1b34ae6
style: Improve actions and workflows style ( #2060 )
2025-04-30 11:55:27 +01:00
Ayaz Salikhov
5860a90b94
fix: Quote release title ( #2064 )
...
Fix: https://github.com/XRPLF/clio/issues/2063
2025-04-30 11:46:44 +01:00
Ayaz Salikhov
f2f342f7c2
ci: Specify concurrency for github workflows ( #2059 )
2025-04-30 11:45:56 +01:00
Ayaz Salikhov
cc1da5afa9
test: Add build_and_test reusable workflow ( #2048 )
2025-04-29 20:28:51 +01:00
Ayaz Salikhov
de055934e1
chore: Use only pre-commit hooks ( #2057 )
...
Co-authored-by: Maria Shodunke <maria-robobug@users.noreply.github.com >
2025-04-29 18:18:42 +01:00
Ayaz Salikhov
0c201ed952
feat: Introduce release_impl.yml workflow to automatically create rel… ( #2043 )
...
…eases
Work on: https://github.com/XRPLF/clio/issues/1779
Features:
- works in PRs (everything but actually pushing a release)
- supports custom release notes header
- removes previous release for nightly releases
The idea is that creating releases should be automatic or almost
automatic and should work more or less the same for nightly and actual
releases.
I haven't yet implemented automatic release process for the actual
release (and don't plan to do it here), but this PR will make it much
easier to implement it in the future.
I suggest deferring discussion of how actual releases should work till
the next PR.
2025-04-29 16:58:25 +01:00
Ayaz Salikhov
9b83eb4033
style: Add C++ pre-commit hooks ( #2039 )
2025-04-29 15:12:38 +01:00
Ayaz Salikhov
fadd60e68a
fix: Don't use steps.conan.outputs.conan_profile as it doesn't exist … ( #2056 )
2025-04-29 13:32:47 +01:00
Ayaz Salikhov
eef85b520a
ci: Use ghcr.io/xrplf/clio-ci docker image ( #2055 )
2025-04-29 11:35:26 +01:00
github-actions[bot]
8b99013ff8
style: clang-tidy auto fixes ( #2053 )
...
Fixes #2052 . Please review and commit clang-tidy fixes.
Co-authored-by: godexsoft <385326+godexsoft@users.noreply.github.com >
Co-authored-by: Alex Kremer <akremer@ripple.com >
Co-authored-by: Ayaz Salikhov <mathbunnyru@users.noreply.github.com >
2025-04-29 10:52:35 +01:00
Ayaz Salikhov
af0a8af287
style: Add typos pre-commit hook ( #2041 )
2025-04-29 09:59:11 +01:00
Ayaz Salikhov
de43be887e
ci: Try to push to ghcr.io ( #2051 )
...
I don't think I can change where it's pushed and start using it in the
same PR, so will try to first update the push part, and then change
where the image is used. Only doing it for CI image to not break
anything.
2025-04-28 19:04:22 +01:00
Ayaz Salikhov
b03cd63efd
fix: Remove Mac CMake installation error ( #2040 )
...
Even on a successful build I see an error from `brew install`
https://github.com/XRPLF/clio/actions/runs/14668965915?pr=2039
2025-04-28 19:00:15 +01:00
Ayaz Salikhov
76cf2fc57e
chore: Disable add_label for conventional commits ( #2054 )
2025-04-28 18:55:26 +01:00
Ayaz Salikhov
f7f619fd1b
chore: Don't use wandalen/wretry.action and update codecov/codecov-ac… ( #2050 )
...
…tion
2025-04-28 18:00:02 +01:00
Ayaz Salikhov
4a0267fbac
style: Add markdownlint pre-commit hook ( #2038 )
...
Co-authored-by: Maria Shodunke <maria-robobug@users.noreply.github.com >
2025-04-28 16:31:07 +01:00
Ayaz Salikhov
b51ed8dd98
fix: Use ytanikin/pr-conventional-commits action for conventional com… ( #2049 )
...
…mits check
There are 2 actions with almost the same name from the same author:
- https://github.com/ytanikin/PRConventionalCommits : the one we use, not
updated for 7 months
- https://github.com/ytanikin/pr-conventional-commits : seems to be more
recent, has more stars and releases
I've merged this change to my develop branch in my fork and created a PR
on to test it works: https://github.com/mathbunnyru/clio/pull/3
I will update the upstream issue and suggest author to archive the old
action: https://github.com/ytanikin/pr-conventional-commits/issues/19
2025-04-28 16:17:23 +01:00
Ayaz Salikhov
cb01c9f596
fix: Add missing paths to update_docker_ci.yml ( #2044 )
2025-04-28 16:06:09 +01:00
Ayaz Salikhov
7c55529c90
chore: Capitalize Dockerfile to add support of tools ( #2045 )
...
For example, there is not syntax highlighting in GitHub, because file is
named `dockerfile` (and not Dockerfile), for example:
https://github.com/XRPLF/clio/blob/release/2.4.0/docker/ci/dockerfile
I also grep'ed dockerfile in repo and fixed some other places
2025-04-28 16:05:37 +01:00
Ayaz Salikhov
424af5dfe0
style: Apply go fmt to go code ( #2046 )
...
~I will add pre-commit hook later if I find a good one~
Found a nice repo, but it is no longer maintained:
https://github.com/dnephin/pre-commit-golang
So, I implemented the check as a local hook.
2025-04-28 16:04:50 +01:00
Ayaz Salikhov
bebc683ba2
chore: Specify conan profile explicitly and don't override it ( #2042 )
2025-04-28 15:00:44 +01:00
Ayaz Salikhov
1e0a2f5162
style: Add prettier pre-commit hook ( #2031 )
...
There are 2 things to know about prettier:
- it's quite pretty most of the time
- it's not configurable
2025-04-25 16:24:45 +01:00
Peter Chen
593d7298b1
fix: Add CTID to all RPC's that includes transactions ( #2011 )
...
fixes #1998
2025-04-25 09:55:39 -04:00
Ayaz Salikhov
79e6f07863
fix: Pin cmake version in ci Dockerfile ( #2036 )
...
Fix: https://github.com/XRPLF/clio/issues/2032
2025-04-25 12:23:44 +01:00
github-actions[bot]
1cb09a01f6
style: clang-tidy auto fixes ( #2034 )
...
Fixes #2033 .
Co-authored-by: godexsoft <385326+godexsoft@users.noreply.github.com >
2025-04-25 11:10:05 +01:00
Ayaz Salikhov
9c92a2b51b
style: Use pre-commit tool and add simple config ( #2029 )
...
I started with really simple pre-commit hooks and will add more on top.
Important files:
- `.pre-commit-config.yaml` - the config for pre-commit
- `.github/workflows/pre-commit.yml` - runs pre-commit hooks in branches
and `develop`
- `.github/workflows/pre-commit-autoupdate.yml` - autoupdates pre-commit
hooks once in a month
2025-04-24 17:59:43 +01:00
dependabot[bot]
99580a2602
ci: [DEPENDABOT] bump docker/build-push-action from 6.15.0 to 6.16.0 in /.github/actions/build_docker_image ( #2030 )
...
Bumps
[docker/build-push-action](https://github.com/docker/build-push-action )
from 6.15.0 to 6.16.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/docker/build-push-action/releases ">docker/build-push-action's
releases</a>.</em></p>
<blockquote>
<h2>v6.16.0</h2>
<ul>
<li>Handle no default attestations env var by <a
href="https://github.com/crazy-max "><code>@crazy-max</code></a> in <a
href="https://redirect.github.com/docker/build-push-action/pull/1343 ">docker/build-push-action#1343</a></li>
<li>Only print secret keys in build summary output by <a
href="https://github.com/crazy-max "><code>@crazy-max</code></a> in <a
href="https://redirect.github.com/docker/build-push-action/pull/1353 ">docker/build-push-action#1353</a></li>
<li>Bump <code>@docker/actions-toolkit</code> from 0.56.0 to 0.59.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1352 ">docker/build-push-action#1352</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.15.0...v6.16.0 ">https://github.com/docker/build-push-action/compare/v6.15.0...v6.16.0 </a></p>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="14487ce63chttps://redirect.github.com/docker/build-push-action/issues/1343 ">#1343</a>
from crazy-max/fix-no-default-attest</li>
<li><a
href="0ec91264d8https://redirect.github.com/docker/build-push-action/issues/1366 ">#1366</a>
from crazy-max/pr-assign-author</li>
<li><a
href="b749522b90c566248492https://redirect.github.com/docker/build-push-action/issues/1363 ">#1363</a>
from crazy-max/fix-codecov</li>
<li><a
href="13275dd76e67dc78bbafhttps://redirect.github.com/docker/build-push-action/issues/1361 ">#1361</a>
from mschoettle/patch-1</li>
<li><a
href="07605044371c198f4467288d9e2e4a88844b95d8https://redirect.github.com/docker/build-push-action/issues/1353 ">#1353</a>
from crazy-max/summary-secret-keys</li>
<li>Additional commits viewable in <a
href="471d1dc4e0...14487ce63chttps://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=docker/build-push-action&package-manager=github_actions&previous-version=6.15.0&new-version=6.16.0 )](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores )
Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)
</details>
Signed-off-by: dependabot[bot] <support@github.com >
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-04-24 17:21:43 +01:00
Ayaz Salikhov
ade6289de2
refactor: Add str() method to BaseTagDecorator ( #2020 )
...
Fix: https://github.com/XRPLF/clio/issues/2008
2025-04-24 16:48:19 +01:00
Ayaz Salikhov
3916635037
chore: Remove dead code related to centos ( #2028 )
2025-04-24 15:26:41 +01:00
dependabot[bot]
f195a30a76
ci: [DEPENDABOT] bump docker/build-push-action from 5.4.0 to 6.15.0 in /.github/actions/build_docker_image ( #2027 )
...
Bumps
[docker/build-push-action](https://github.com/docker/build-push-action )
from 5.4.0 to 6.15.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/docker/build-push-action/releases ">docker/build-push-action's
releases</a>.</em></p>
<blockquote>
<h2>v6.15.0</h2>
<ul>
<li>Bump <code>@docker/actions-toolkit</code> from 0.55.0 to 0.56.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1330 ">docker/build-push-action#1330</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.14.0...v6.15.0 ">https://github.com/docker/build-push-action/compare/v6.14.0...v6.15.0 </a></p>
<h2>v6.14.0</h2>
<ul>
<li>Bump <code>@docker/actions-toolkit</code> from 0.53.0 to 0.55.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1324 ">docker/build-push-action#1324</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.13.0...v6.14.0 ">https://github.com/docker/build-push-action/compare/v6.13.0...v6.14.0 </a></p>
<h2>v6.13.0</h2>
<ul>
<li>Bump <code>@docker/actions-toolkit</code> from 0.51.0 to 0.53.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1308 ">docker/build-push-action#1308</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.12.0...v6.13.0 ">https://github.com/docker/build-push-action/compare/v6.12.0...v6.13.0 </a></p>
<h2>v6.12.0</h2>
<ul>
<li>Bump <code>@docker/actions-toolkit</code> from 0.49.0 to 0.51.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1300 ">docker/build-push-action#1300</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.11.0...v6.12.0 ">https://github.com/docker/build-push-action/compare/v6.11.0...v6.12.0 </a></p>
<h2>v6.11.0</h2>
<ul>
<li>Handlebar <code>defaultContext</code> support for
<code>build-contexts</code> input by <a
href="https://github.com/crazy-max "><code>@crazy-max</code></a> in <a
href="https://redirect.github.com/docker/build-push-action/pull/1283 ">docker/build-push-action#1283</a></li>
<li>Bump <code>@docker/actions-toolkit</code> from 0.46.0 to 0.49.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1281 ">docker/build-push-action#1281</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.10.0...v6.11.0 ">https://github.com/docker/build-push-action/compare/v6.10.0...v6.11.0 </a></p>
<h2>v6.10.0</h2>
<ul>
<li>Add <code>call</code> input to set method for evaluating build by <a
href="https://github.com/crazy-max "><code>@crazy-max</code></a> in <a
href="https://redirect.github.com/docker/build-push-action/pull/1265 ">docker/build-push-action#1265</a></li>
<li>Bump <code>@actions/core</code> from 1.10.1 to 1.11.1 in <a
href="https://redirect.github.com/docker/build-push-action/pull/1238 ">docker/build-push-action#1238</a></li>
<li>Bump <code>@docker/actions-toolkit</code> from 0.39.0 to 0.46.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1268 ">docker/build-push-action#1268</a></li>
<li>Bump cross-spawn from 7.0.3 to 7.0.6 in <a
href="https://redirect.github.com/docker/build-push-action/pull/1261 ">docker/build-push-action#1261</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.9.0...v6.10.0 ">https://github.com/docker/build-push-action/compare/v6.9.0...v6.10.0 </a></p>
<h2>v6.9.0</h2>
<ul>
<li>Bump <code>@docker/actions-toolkit</code> from 0.38.0 to 0.39.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1234 ">docker/build-push-action#1234</a></li>
<li>Bump path-to-regexp from 6.2.2 to 6.3.0 in <a
href="https://redirect.github.com/docker/build-push-action/pull/1232 ">docker/build-push-action#1232</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.8.0...v6.9.0 ">https://github.com/docker/build-push-action/compare/v6.8.0...v6.9.0 </a></p>
<h2>v6.8.0</h2>
<ul>
<li>Bump <code>@docker/actions-toolkit</code> from 0.37.1 to 0.38.0 in
<a
href="https://redirect.github.com/docker/build-push-action/pull/1230 ">docker/build-push-action#1230</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.7.0...v6.8.0 ">https://github.com/docker/build-push-action/compare/v6.7.0...v6.8.0 </a></p>
<h2>v6.7.0</h2>
<ul>
<li>Print info message for build summary support checks by <a
href="https://github.com/crazy-max "><code>@crazy-max</code></a> in <a
href="https://redirect.github.com/docker/build-push-action/pull/1211 ">docker/build-push-action#1211</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a
href="https://github.com/docker/build-push-action/compare/v6.6.1...v6.7.0 ">https://github.com/docker/build-push-action/compare/v6.6.1...v6.7.0 </a></p>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="471d1dc4e0https://redirect.github.com/docker/build-push-action/issues/1330 ">#1330</a>
from docker/dependabot/npm_and_yarn/docker/actions-t...</li>
<li><a
href="b89ff0a6f21e3ae3a4d3b16f42f92ahttps://redirect.github.com/docker/build-push-action/issues/1325 ">#1325</a>
from crazy-max/buildx-edge</li>
<li><a
href="dc0fea5e620adf995921https://redirect.github.com/docker/build-push-action/issues/1324 ">#1324</a>
from docker/dependabot/npm_and_yarn/docker/actions-t...</li>
<li><a
href="d88cd289df3d09a6bd70ca877d9245https://redirect.github.com/docker/build-push-action/issues/1308 ">#1308</a>
from docker/dependabot/npm_and_yarn/docker/actions-t...</li>
<li><a
href="d2fe919bb5ca052bb54a...471d1dc4e0https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=docker/build-push-action&package-manager=github_actions&previous-version=5.4.0&new-version=6.15.0 )](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores )
Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)
</details>
Signed-off-by: dependabot[bot] <support@github.com >
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-04-24 12:51:08 +01:00
Ayaz Salikhov
cc610a0964
ci: Fix time field to be string in dependabot config ( #2024 )
...
Sorry, it seems to have been the wrong change.
I pushed this to `develop` in my fork, enabled Dependabot there and PRs
were created in my fork.
Close: https://github.com/XRPLF/clio/issues/2022
2025-04-24 12:19:44 +01:00
github-actions[bot]
c5012eb854
style: clang-tidy auto fixes ( #2026 )
...
Fixes #2025 .
Co-authored-by: godexsoft <385326+godexsoft@users.noreply.github.com >
2025-04-24 11:19:28 +01:00
Ayaz Salikhov
01ae4c4312
refactor: Compare error_code with enum values, not hardcoded ints ( #2023 )
...
Fix: https://github.com/XRPLF/clio/issues/2002
2025-04-23 18:39:53 +01:00
Ayaz Salikhov
88d27a7265
chore: Pin external actions using hashes ( #2019 )
...
Note: I used exactly the version which are already being used (didn't
update anything).
Updates will come when we merge https://github.com/XRPLF/clio/pull/2018
2025-04-23 17:35:40 +01:00
Ayaz Salikhov
e01d7d12cf
chore: Run dependabot for all local actions ( #2018 )
2025-04-23 16:54:32 +01:00
github-actions[bot]
1e38ad5ec0
style: clang-tidy auto fixes ( #2013 )
...
Fixes #2012 . Please review and commit clang-tidy fixes.
Co-authored-by: godexsoft <385326+godexsoft@users.noreply.github.com >
2025-04-21 10:26:46 -04:00
Sergey Kuznetsov
bd9e39ee85
fix: Fix incorrect requests logging ( #2005 )
...
Fixes #2004 .
2025-04-17 17:49:44 +01:00
Sergey Kuznetsov
46514c8fe9
feat: More efficient cache ( #1997 )
...
Fixes #1473 .
2025-04-17 16:44:53 +01:00
