Skip to content

Commit ac0e56c

Browse files
mukhrrmukhrr
committed
optimistically update Approved and First Approved column in offline
2 parents dba25ee + 534061a commit ac0e56c

279 files changed

Lines changed: 4703 additions & 2615 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.claude/settings.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
"hooks": [
77
{
88
"type": "command",
9-
"command": "FILE=$(jq -r '.tool_input.file_path') && [ -f \"$FILE\" ] && ./node_modules/.bin/prettier --experimental-cli --no-cache --write --ignore-unknown \"$FILE\""
9+
"command": "FILE=$(jq -r '.tool_input.file_path') && [ -f \"$FILE\" ] && ./node_modules/.bin/oxfmt --write --no-error-on-unmatched-pattern \"$FILE\""
1010
}
1111
]
1212
}

‎.github/actions/composite/setupNode/action.yml‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ description: Set up Node
33

44
inputs:
55
IS_HYBRID_BUILD:
6-
description: "Indicates if node is set up for hybrid app"
6+
description: 'Indicates if node is set up for hybrid app'
77
required: false
88
default: 'false'
99

@@ -33,7 +33,7 @@ runs:
3333
with:
3434
path: node_modules
3535
key: ${{ inputs.IS_HYBRID_BUILD == 'true' && format('{0}-node-modules-{1}', runner.os, hashFiles('package-lock.json', 'patches/**', 'Mobile-Expensify/patches/**')) || format('{0}-node-modules-{1}', runner.os, hashFiles('package-lock.json', 'patches/**'))}}
36-
36+
3737
- id: cache-old-dot-node-modules
3838
if: inputs.IS_HYBRID_BUILD == 'true'
3939
# v5.0.1
@@ -48,9 +48,9 @@ runs:
4848
run: rm -rf node_modules
4949

5050
- name: Install root project node packages
51-
if: steps.cache-node-modules.outputs.cache-hit != 'true' || (inputs.IS_HYBRID_BUILD == 'true' && steps.cache-old-dot-node-modules.outputs.cache-hit != 'true')
51+
if: steps.cache-node-modules.outputs.cache-hit != 'true' || (inputs.IS_HYBRID_BUILD == 'true' && steps.cache-old-dot-node-modules.outputs.cache-hit != 'true')
5252
uses: nick-fields/retry@ad984534de44a9489a53aefd81eb77f87c70dc60 # v4
5353
with:
5454
timeout_minutes: 30
5555
max_attempts: 3
56-
command: npm ci
56+
command: npm ci

‎.github/actions/javascript/bumpVersion/action.yml‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
name: Bump npm version
22
description: Increase the application version (JS and native), based on git tags
33
inputs:
4-
SEMVER_LEVEL:
5-
description: Semantic Versioning Level
6-
required: true
4+
SEMVER_LEVEL:
5+
description: Semantic Versioning Level
6+
required: true
77
outputs:
8-
NEW_VERSION:
9-
description: The new semver version of the application, updated in the JS and native layers.
8+
NEW_VERSION:
9+
description: The new semver version of the application, updated in the JS and native layers.
1010
runs:
11-
using: node24
12-
main: ./index.js
11+
using: node24
12+
main: ./index.js

‎.github/actions/javascript/checkAndroidStatus/action.yml‎

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,22 @@
11
name: 'Check Android Status'
22
description: 'Checks the status of the Android track and calculates the rollout percentage.'
33
inputs:
4-
GITHUB_TOKEN:
5-
description: Auth token for New Expensify Github
6-
required: true
7-
GOOGLE_KEY_FILE:
8-
description: Authentication file for Google Cloud API
9-
required: true
10-
PACKAGE_NAME:
11-
description: Package name to check the status of
12-
required: true
4+
GITHUB_TOKEN:
5+
description: Auth token for New Expensify Github
6+
required: true
7+
GOOGLE_KEY_FILE:
8+
description: Authentication file for Google Cloud API
9+
required: true
10+
PACKAGE_NAME:
11+
description: Package name to check the status of
12+
required: true
1313
outputs:
14-
HALTED:
15-
description: True if the app is halted, false otherwise
16-
COMPLETED:
17-
description: True if the app is completed a rollout, false otherwise
18-
ROLLOUT_PERCENTAGE:
19-
description: The calculated rollout percentage
14+
HALTED:
15+
description: True if the app is halted, false otherwise
16+
COMPLETED:
17+
description: True if the app is completed a rollout, false otherwise
18+
ROLLOUT_PERCENTAGE:
19+
description: The calculated rollout percentage
2020
runs:
21-
using: 'node24'
22-
main: './index.js'
21+
using: 'node24'
22+
main: './index.js'

‎.github/actions/javascript/formatCodeCovComment/action.yml‎

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,9 @@
1-
name: "Format CodeCov Comment"
2-
description: "Reformats CodeCov PR comments to display only relevant information"
1+
name: 'Format CodeCov Comment'
2+
description: 'Reformats CodeCov PR comments to display only relevant information'
33
inputs:
4-
GITHUB_TOKEN:
5-
description: "Github token for authentication"
6-
required: true
4+
GITHUB_TOKEN:
5+
description: 'Github token for authentication'
6+
required: true
77
runs:
8-
using: "node24"
9-
main: "./index.js"
10-
8+
using: 'node24'
9+
main: './index.js'

‎.github/actions/javascript/getAndroidRolloutPercentage/action.yml‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
name: 'Get Android Rollout percentage'
22
description: 'Gets the current Android track rollout percentage.'
33
inputs:
4-
GOOGLE_KEY_FILE:
5-
description: Authentication file for Google Cloud API
6-
required: true
7-
PACKAGE_NAME:
8-
description: Package name to check the status of
9-
required: true
4+
GOOGLE_KEY_FILE:
5+
description: Authentication file for Google Cloud API
6+
required: true
7+
PACKAGE_NAME:
8+
description: Package name to check the status of
9+
required: true
1010
outputs:
11-
CURRENT_ROLLOUT_PERCENTAGE:
12-
description: 'The current rollout percentage of the track'
11+
CURRENT_ROLLOUT_PERCENTAGE:
12+
description: 'The current rollout percentage of the track'
1313
runs:
14-
using: 'node24'
15-
main: './index.js'
14+
using: 'node24'
15+
main: './index.js'

‎.github/actions/javascript/getArtifactInfo/action.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,5 +18,5 @@ outputs:
1818
description: The ID of the workflow that produced the artifact
1919

2020
runs:
21-
using: "node24"
22-
main: "index.js"
21+
using: 'node24'
22+
main: 'index.js'

‎.github/actions/javascript/getDeployPullRequestList/action.yml‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,20 @@
11
name: 'Get Release Pull Request List'
22
description: 'Gather all the PRs being deployed for given release tag.'
33
inputs:
4-
TAG:
5-
description: Git tag
6-
required: true
7-
GITHUB_TOKEN:
8-
description: "Github token for authentication"
9-
required: true
10-
IS_PRODUCTION_DEPLOY:
11-
description: "True if we are deploying to production"
12-
required: false
4+
TAG:
5+
description: Git tag
6+
required: true
7+
GITHUB_TOKEN:
8+
description: 'Github token for authentication'
9+
required: true
10+
IS_PRODUCTION_DEPLOY:
11+
description: 'True if we are deploying to production'
12+
required: false
1313
outputs:
14-
PR_LIST:
15-
description: Array of pull request numbers
16-
MOBILE_EXPENSIFY_PR_LIST:
17-
description: Array of Mobile-Expensify pull request numbers
14+
PR_LIST:
15+
description: Array of pull request numbers
16+
MOBILE_EXPENSIFY_PR_LIST:
17+
description: Array of Mobile-Expensify pull request numbers
1818
runs:
19-
using: 'node24'
20-
main: './index.js'
19+
using: 'node24'
20+
main: './index.js'

‎.github/actions/javascript/getPullRequestIncrementalChanges/action.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
name: 'Check Pull Request Incremental Changes'
22
description: |
33
Detects actual file changes in PR opened or synchronize events, ignoring false positives from merge commits.
4-
4+
55
This action is specifically designed for pull_request synchronize events where you need to distinguish
66
between actual changes pushed to the PR versus changes that came from merging the base branch.
7-
7+
88
For simple file change detection in PRs, use `gh pr diff <pr_number>` instead.
99
1010
inputs:

‎.github/actions/javascript/isAuthorizedContributor/action.yml‎

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,27 @@
11
name: 'Is Authorized Contributor'
22
description: 'Checks whether a PR author is authorized to contribute (org member, Contributor+, or linked issue)'
33
inputs:
4-
PR_NUMBER:
5-
description: Pull request number
6-
required: true
7-
ACTOR:
8-
description: Login of the user to authorize (PR author, comment author, etc.)
9-
required: true
10-
ACTOR_ASSOCIATION:
11-
description: "Actor's association with the repository (MEMBER, OWNER, CONTRIBUTOR, etc.)"
12-
required: true
13-
GITHUB_TOKEN:
14-
description: Auth token for repository API calls
15-
required: true
16-
default: ${{ github.token }}
17-
OS_BOTIFY_TOKEN:
18-
description: OSBotify token for org team membership (read:org)
19-
required: true
4+
PR_NUMBER:
5+
description: Pull request number
6+
required: true
7+
ACTOR:
8+
description: Login of the user to authorize (PR author, comment author, etc.)
9+
required: true
10+
ACTOR_ASSOCIATION:
11+
description: "Actor's association with the repository (MEMBER, OWNER, CONTRIBUTOR, etc.)"
12+
required: true
13+
GITHUB_TOKEN:
14+
description: Auth token for repository API calls
15+
required: true
16+
default: ${{ github.token }}
17+
OS_BOTIFY_TOKEN:
18+
description: OSBotify token for org team membership (read:org)
19+
required: true
2020
outputs:
21-
IS_AUTHORIZED:
22-
description: "'true' if the contributor is authorized, 'false' otherwise"
23-
IS_INTERNAL:
24-
description: "'true' if the actor is an internal Expensify engineer (engineering org team member), 'false' otherwise"
21+
IS_AUTHORIZED:
22+
description: "'true' if the contributor is authorized, 'false' otherwise"
23+
IS_INTERNAL:
24+
description: "'true' if the actor is an internal Expensify engineer (engineering org team member), 'false' otherwise"
2525
runs:
26-
using: 'node24'
27-
main: './index.js'
26+
using: 'node24'
27+
main: './index.js'

0 commit comments

Comments
 (0)