-
Notifications
You must be signed in to change notification settings - Fork 51
Merge in develop #1656
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
Merge in develop #1656
Changes from all commits
f9cf766
4198859
26274ce
25754fe
7ca8c9a
32c1a42
ef8acb0
409270d
04fc9ec
158140b
8a256a0
2cc9ba2
356f008
2c40ccd
4bb611f
9d68f4f
973049f
c0ec5fb
ab9bb07
49c655e
a745f7b
86e5ca3
22cd966
4e47782
5f7bbad
ab260d4
339342a
babaa8e
190f97e
8f5bca9
cfe0b83
bf96af0
c41bf69
c2f2a0a
02d1dc4
1f72e1d
6ab92b8
212809d
926dab0
eff3ee3
f5feace
ead28d2
b419b66
4968fc3
dc6a09a
971f208
a1ff894
596b8c4
caebce7
6434996
acac808
713ade8
b49d3eb
ce462b5
f9e5ac1
aa01e3d
16437c8
ffaa101
5aaff3d
ff855cf
a70e1a5
4d24155
6a43f63
619b5c0
1dee45f
a0fe205
374be0c
c7d59d7
1b719fb
441e6d7
592a0f3
4fd1543
81a1173
06ab559
90f0395
41329e1
7eccfe3
18768fd
592b787
5823808
b184e86
125cf7d
8af3cf0
9605fa2
530c136
3ddfca7
cf7491a
eedc3ff
ac1de1a
cf73dbc
392e5f4
de59094
52fb0ab
91b2eb2
d907e6e
7c2a793
8255022
e6f13e0
de8f892
21d0574
2422565
5334423
7d0fcd9
5431440
3b7fbe2
bbe6312
997d8f0
3c464c0
c8c1075
f39bef3
67d387c
70f514e
e04fae1
50d6f0a
b0d0f48
9f1728f
00f7089
879c08c
0ebe42d
3cf1900
0493017
d673228
f089527
1fc4c52
4ec89b1
6b81b05
cd5322c
1e3c8e0
0eb4fb3
8390251
713528f
a211634
6063bbe
f489bae
0b819d7
fc93578
9c67f2b
3b67a7e
153ab96
1aa70f4
78755ed
b3eb22c
612a798
8d4d3a8
37c544a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
name: AI PR Reviewer | ||
|
||
on: | ||
pull_request: | ||
types: | ||
- opened | ||
- synchronize | ||
permissions: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The |
||
pull-requests: write | ||
jobs: | ||
tc-ai-pr-review: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout Repo | ||
uses: actions/checkout@v3 | ||
|
||
- name: TC AI PR Reviewer | ||
uses: topcoder-platform/tc-ai-pr-reviewer@master | ||
with: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # The GITHUB_TOKEN is there by default so you just need to keep it like it is and not necessarily need to add it as secret as it will throw an error. [More Details](https://docs.github.com/en/actions/security-guides/automatic-token-authentication#about-the-github_token-secret) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider clarifying the comment about the |
||
LAB45_API_KEY: ${{ secrets.LAB45_API_KEY }} | ||
exclude: "**/*.json, **/*.md, **/*.jpg, **/*.png, **/*.jpeg, **/*.bmp, **/*.webp" # Optional: exclude patterns separated by commas |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -19,12 +19,15 @@ module.exports = { | |
CHALLENGE_TRACKS_URL: `${DEV_API_HOSTNAME}/v5/challenge-tracks`, | ||
CHALLENGE_PHASES_URL: `${DEV_API_HOSTNAME}/v5/challenge-phases`, | ||
CHALLENGE_TIMELINES_URL: `${DEV_API_HOSTNAME}/v5/challenge-timelines`, | ||
COPILOTS_URL: `https://copilots.${DOMAIN}`, | ||
PROJECT_API_URL: `${DEV_API_HOSTNAME}/v5/projects`, | ||
GROUPS_API_URL: `${DEV_API_HOSTNAME}/v5/groups`, | ||
TERMS_API_URL: `${DEV_API_HOSTNAME}/v5/terms`, | ||
MEMBERS_API_URL: `${DEV_API_HOSTNAME}/v5/members`, | ||
RESOURCES_API_URL: `${DEV_API_HOSTNAME}/v5/resources`, | ||
RESOURCE_ROLES_API_URL: `${DEV_API_HOSTNAME}/v5/resource-roles`, | ||
SUBMISSIONS_API_URL: `${DEV_API_HOSTNAME}/v5/submissions`, | ||
REVIEW_TYPE_API_URL: `${DEV_API_HOSTNAME}/v5/reviewTypes`, | ||
SUBMISSION_REVIEW_APP_URL: `https://submission-review.${DOMAIN}/challenges`, | ||
STUDIO_URL: `https://studio.${DOMAIN}`, | ||
CONNECT_APP_URL: `https://connect.${DOMAIN}`, | ||
|
@@ -44,8 +47,10 @@ module.exports = { | |
CREATE_FORUM_TYPE_IDS: ['927abff4-7af9-4145-8ba1-577c16e64e2e', 'dc876fa4-ef2d-4eee-b701-b555fcc6544c', 'ecd58c69-238f-43a4-a4bb-d172719b9f31', '78b37a69-92d5-4ad7-bf85-c79b65420c79', '929bc408-9cf2-4b3e-ba71-adfbf693046c'], | ||
FILE_PICKER_API_KEY: process.env.FILE_PICKER_API_KEY, | ||
FILE_PICKER_CONTAINER_NAME: 'tc-challenge-v5-dev', | ||
FILE_PICKER_SUBMISSION_CONTAINER_NAME: process.env.FILE_PICKER_SUBMISSION_CONTAINER_NAME || 'submission-staging-dev', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider validating the |
||
FILE_PICKER_REGION: 'us-east-1', | ||
FILE_PICKER_CNAME: 'fs.topcoder.com', | ||
FILE_PICKER_LOCATION: 's3', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The addition of |
||
// if idle for this many minutes, show user a prompt saying they'll be logged out | ||
IDLE_TIMEOUT_MINUTES: 10, | ||
// duration to show the prompt saying user will be logged out, before actually logging out the user | ||
|
@@ -57,5 +62,5 @@ module.exports = { | |
SKILLS_V5_API_URL: `${API_V5}/standardized-skills/skills/autocomplete`, | ||
UPDATE_SKILLS_V5_API_URL: `${API_V5}/standardized-skills/challenge-skills`, | ||
SALESFORCE_BILLING_ACCOUNT_LINK: 'https://c.cs18.visual.force.com/apex/baredirect?id=', | ||
TYPEFORM_URL: 'https://topcoder.typeform.com/to/YJ7AL4p8' | ||
PROFILE_URL: 'https://profiles.topcoder-dev.com/' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The |
||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,12 +18,15 @@ module.exports = { | |
CHALLENGE_TRACKS_URL: `${PROD_API_HOSTNAME}/v5/challenge-tracks`, | ||
CHALLENGE_PHASES_URL: `${PROD_API_HOSTNAME}/v5/challenge-phases`, | ||
CHALLENGE_TIMELINES_URL: `${PROD_API_HOSTNAME}/v5/challenge-timelines`, | ||
COPILOTS_URL: `https://copilots.${DOMAIN}`, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ensure that |
||
PROJECT_API_URL: `${PROD_API_HOSTNAME}/v5/projects`, | ||
GROUPS_API_URL: `${PROD_API_HOSTNAME}/v5/groups`, | ||
TERMS_API_URL: `${PROD_API_HOSTNAME}/v5/terms`, | ||
MEMBERS_API_URL: `${PROD_API_HOSTNAME}/v5/members`, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Verify that the endpoint |
||
RESOURCES_API_URL: `${PROD_API_HOSTNAME}/v5/resources`, | ||
RESOURCE_ROLES_API_URL: `${PROD_API_HOSTNAME}/v5/resource-roles`, | ||
SUBMISSIONS_API_URL: `${PROD_API_HOSTNAME}/v5/submissions`, | ||
REVIEW_TYPE_API_URL: `${PROD_API_HOSTNAME}/v5/reviewTypes`, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Check that the |
||
SUBMISSION_REVIEW_APP_URL: `https://submission-review.${DOMAIN}/challenges`, | ||
STUDIO_URL: `https://studio.${DOMAIN}`, | ||
CONNECT_APP_URL: `https://connect.${DOMAIN}`, | ||
|
@@ -43,8 +46,10 @@ module.exports = { | |
CREATE_FORUM_TYPE_IDS: ['927abff4-7af9-4145-8ba1-577c16e64e2e', 'dc876fa4-ef2d-4eee-b701-b555fcc6544c', 'ecd58c69-238f-43a4-a4bb-d172719b9f31', '78b37a69-92d5-4ad7-bf85-c79b65420c79', '929bc408-9cf2-4b3e-ba71-adfbf693046c'], | ||
FILE_PICKER_API_KEY: process.env.FILE_PICKER_API_KEY, | ||
FILE_PICKER_CONTAINER_NAME: 'tc-challenge-v5-prod', | ||
FILE_PICKER_SUBMISSION_CONTAINER_NAME: process.env.FILE_PICKER_SUBMISSION_CONTAINER_NAME || 'submission-staging-prod', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider verifying that the default value |
||
FILE_PICKER_REGION: 'us-east-1', | ||
FILE_PICKER_CNAME: 'fs.topcoder.com', | ||
FILE_PICKER_LOCATION: 's3', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ensure that the value |
||
IDLE_TIMEOUT_MINUTES: 10, | ||
IDLE_TIMEOUT_GRACE_MINUTES: 5, | ||
MULTI_ROUND_CHALLENGE_TEMPLATE_ID: 'd4201ca4-8437-4d63-9957-3f7708184b07', | ||
|
@@ -54,5 +59,5 @@ module.exports = { | |
SKILLS_V5_API_URL: `${API_V5}/standardized-skills/skills/autocomplete`, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The |
||
UPDATE_SKILLS_V5_API_URL: `${API_V5}/standardized-skills/challenge-skills`, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The |
||
SALESFORCE_BILLING_ACCOUNT_LINK: 'https://topcoder.my.salesforce.com/apex/baredirect?id=', | ||
TYPEFORM_URL: 'https://topcoder.typeform.com/to/YJ7AL4p8' | ||
PROFILE_URL: 'https://profiles.topcoder.com/' | ||
} |
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.
The addition of the
-p FARGATE
parameter to the./master_deploy.sh
command changes the deployment platform. Ensure that themaster_deploy.sh
script is updated to handle this new parameter correctly, and verify that FARGATE is supported in the deployment context.