👷(CLDSRV-860) Monitor async await migration by DarkIsDude · Pull Request #6088 · scality/cloudserver

DarkIsDude · 2026-02-25T15:47:58Z

https://scality.atlassian.net/wiki/spaces/OS/pages/3523346468/2025-10-30+-+Async+Await+migration

.github/scripts/check-diff-async.mjs

.github/workflows/tests.yaml

.github/scripts/count-async-functions.mjs

.github/scripts/check-diff-async.mjs

.github/workflows/tests.yaml

.github/scripts/count-async-functions.mjs

.github/workflows/tests.yaml

.github/scripts/check-diff-async.mjs

.github/scripts/count-async-functions.mjs

.github/scripts/check-diff-async.mjs

.github/scripts/count-async-functions.mjs

.github/scripts/check-diff-async.mjs

.github/scripts/count-async-functions.mjs

.github/scripts/check-diff-async.mjs

.github/scripts/count-async-functions.mjs

DarkIsDude · 2026-04-13T12:21:22Z

/approve

bert-e · 2026-04-13T12:21:33Z

Integration data created

I have created the integration data for the additional destination branches.

this pull request will merge feature/CLDSRV-860/monitor-async-await-migration into
development/9.3
w/9.4/feature/CLDSRV-860/monitor-async-await-migration will be merged into development/9.4

The following branches will NOT be impacted:

development/7.10
development/7.4
development/7.70
development/8.8
development/9.0
development/9.1
development/9.2

You can set option create_pull_requests if you need me to create
integration pull requests in addition to integration branches, with:

@bert-e create_pull_requests

The following options are set: approve

bert-e · 2026-04-13T12:21:36Z

Build failed

The build for commit did not succeed in branch feature/CLDSRV-860/monitor-async-await-migration

The following options are set: approve

claude · 2026-04-13T12:39:01Z

LGTM

Review by Claude Code

Issue: CLDSRV-860

bert-e · 2026-04-13T12:43:55Z

History mismatch

Merge commit #9dff1c59872a7482d6a5c79623a7025071f981f3 on the integration branch
w/9.4/feature/CLDSRV-860/monitor-async-await-migration is merging a branch which is neither the current
branch feature/CLDSRV-860/monitor-async-await-migration nor the development branch
development/9.4.

It is likely due to a rebase of the branch feature/CLDSRV-860/monitor-async-await-migration and the
merge is not possible until all related w/* branches are deleted or updated.

Please use the reset command to have me reinitialize these branches.

The following options are set: approve

claude · 2026-04-13T12:47:19Z

LGTM

Review by Claude Code

bert-e · 2026-04-13T12:47:31Z

Integration data created

I have created the integration data for the additional destination branches.

this pull request will merge feature/CLDSRV-860/monitor-async-await-migration into
development/9.3
w/9.4/feature/CLDSRV-860/monitor-async-await-migration will be merged into development/9.4

The following branches will NOT be impacted:

development/7.10
development/7.4
development/7.70
development/8.8
development/9.0
development/9.1
development/9.2

You can set option create_pull_requests if you need me to create
integration pull requests in addition to integration branches, with:

@bert-e create_pull_requests

The following options are set: approve

bert-e · 2026-04-13T13:09:59Z

Build failed

The build for commit did not succeed in branch feature/CLDSRV-860/monitor-async-await-migration

The following options are set: approve

DarkIsDude self-assigned this Feb 25, 2026

This comment was marked as resolved.

Sign in to view

DarkIsDude changed the base branch from development/9.2 to development/9.3 February 25, 2026 15:48

This comment was marked as resolved.

Sign in to view

DarkIsDude commented Mar 10, 2026

View reviewed changes

.github/scripts/check-diff-async.mjs Outdated Show resolved Hide resolved

DarkIsDude marked this pull request as ready for review March 10, 2026 16:09

DarkIsDude requested review from a team, benzekrimaha and delthas March 10, 2026 16:09

DarkIsDude force-pushed the feature/CLDSRV-860/monitor-async-await-migration branch from 93020af to 8981239 Compare March 10, 2026 16:10

claude bot reviewed Mar 10, 2026

View reviewed changes

.github/scripts/check-diff-async.mjs Outdated Show resolved Hide resolved

claude bot reviewed Mar 10, 2026

View reviewed changes

.github/workflows/tests.yaml Outdated Show resolved Hide resolved

claude bot reviewed Mar 10, 2026

View reviewed changes

.github/scripts/count-async-functions.mjs Outdated Show resolved Hide resolved

claude bot reviewed Mar 10, 2026

View reviewed changes

.github/scripts/check-diff-async.mjs Outdated Show resolved Hide resolved

claude bot reviewed Mar 10, 2026

View reviewed changes

.github/workflows/tests.yaml Show resolved Hide resolved

This comment was marked as resolved.

Sign in to view

francoisferrand requested changes Mar 11, 2026

View reviewed changes

This comment was marked as resolved.

Sign in to view

DarkIsDude force-pushed the feature/CLDSRV-860/monitor-async-await-migration branch from a64c535 to f1e7018 Compare March 19, 2026 13:23

claude bot reviewed Mar 19, 2026

View reviewed changes

.github/scripts/count-async-functions.mjs Outdated Show resolved Hide resolved

claude bot reviewed Mar 19, 2026

View reviewed changes

.github/scripts/count-async-functions.mjs Outdated Show resolved Hide resolved

claude bot reviewed Mar 19, 2026

View reviewed changes

.github/scripts/count-async-functions.mjs Show resolved Hide resolved

claude bot reviewed Mar 19, 2026

View reviewed changes

.github/scripts/check-diff-async.mjs Outdated Show resolved Hide resolved

claude bot reviewed Mar 19, 2026

View reviewed changes

.github/scripts/check-diff-async.mjs Outdated Show resolved Hide resolved

claude bot reviewed Mar 19, 2026

View reviewed changes

.github/scripts/count-async-functions.mjs Outdated Show resolved Hide resolved

This comment was marked as resolved.

Sign in to view

claude bot reviewed Mar 31, 2026

View reviewed changes

.github/scripts/check-diff-async.mjs Show resolved Hide resolved

claude bot reviewed Mar 31, 2026

View reviewed changes

.github/scripts/count-async-functions.mjs Show resolved Hide resolved

This comment was marked as resolved.

Sign in to view

DarkIsDude force-pushed the feature/CLDSRV-860/monitor-async-await-migration branch from 13dcaeb to 2421cbf Compare March 31, 2026 15:13

claude bot reviewed Mar 31, 2026

View reviewed changes

.github/scripts/check-diff-async.mjs Show resolved Hide resolved

claude bot reviewed Mar 31, 2026

View reviewed changes

.github/scripts/count-async-functions.mjs Show resolved Hide resolved

claude bot reviewed Mar 31, 2026

View reviewed changes

.github/scripts/count-async-functions.mjs Show resolved Hide resolved

This comment was marked as resolved.

Sign in to view

DarkIsDude requested a review from francoisferrand March 31, 2026 17:42

francoisferrand approved these changes Apr 3, 2026

View reviewed changes

DarkIsDude added 11 commits April 13, 2026 14:43

🚨 add eslint configuration to monitor async/await

08265a7

Issue: CLDSRV-860

👷 update CI to add new check on async/await file

9623ae1

Issue: CLDSRV-860

👷 remove then check from the code

f7e599c

Issue: CLDSRV-860

👷 remove async/await rule in the CI

1899244

Issue: CLDSRV-860

✨ add count array source path from package.json

9fca0ae

Issue: CLDSRV-860

✅ create test for new scripts

380abb7

Issue: CLDSRV-860

🔒️ use array argument to avoid injection

5ac3a2a

Issue: CLDSRV-860

👷 change how we compute the migration progress

c4be965

Issue: CLDSRV-860

👷 init the CodeQL for async/await

429ded3

Issue: CLDSRV-860

👷 restrict counter only on dev branch

d134d53

Issue: CLDSRV-860

⬆️ upgrade ts-morph to latest version

84461c0

Issue: CLDSRV-860

DarkIsDude force-pushed the feature/CLDSRV-860/monitor-async-await-migration branch from 90e8107 to 84461c0 Compare April 13, 2026 12:43

Conversation

DarkIsDude commented Feb 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as resolved.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment was marked as resolved.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment was marked as resolved.

This comment was marked as resolved.

This comment was marked as resolved.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

This comment was marked as resolved.

Uh oh!

Uh oh!

This comment was marked as resolved.

Uh oh!

Uh oh!

Uh oh!

This comment was marked as resolved.

DarkIsDude commented Apr 13, 2026

Uh oh!

bert-e commented Apr 13, 2026

Integration data created

Uh oh!

bert-e commented Apr 13, 2026

Build failed

Uh oh!

claude bot commented Apr 13, 2026

Uh oh!

bert-e commented Apr 13, 2026

History mismatch

Uh oh!

claude bot commented Apr 13, 2026

Uh oh!

bert-e commented Apr 13, 2026

Integration data created

Uh oh!

bert-e commented Apr 13, 2026

Build failed

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

DarkIsDude commented Feb 25, 2026 •

edited

Loading