-
Notifications
You must be signed in to change notification settings - Fork 6
[PROD RELEASE] - Updates & fixes #42
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
Open
kkartunov
wants to merge
41
commits into
master
Choose a base branch
from
develop
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 38 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
103c3bd
Potential fix for code scanning alert no. 72: Incomplete string escap…
kkartunov 93e0da8
pnpm update --latest
kkartunov 0ffaffb
on dev env
kkartunov 0d99cea
uuid API version updates
kkartunov b8ccdfa
axios-retry api update
kkartunov 663e1e8
joi & express fixes
kkartunov 6530304
fix error, schema.validate
kkartunov 611ee56
clean up readmes
kkartunov de2bc0d
remove unused
kkartunov d4aa9b4
up aws sdk to v3
kkartunov 127cce1
hid pnpm-lock
kkartunov 29ae9d7
Merge pull request #31 from topcoder-platform/alert-autofix-72
kkartunov 972cb96
Merge pull request #32 from topcoder-platform/module-updates
kkartunov d464096
drop yarn
kkartunov 67d72d4
Merge pull request #33 from topcoder-platform/module-updates
kkartunov f5914e2
Fix for processing phases in challenge update
jmgasper 85ff0c7
Merge pull request #34 from topcoder-platform/PS-441
jmgasper bc505af
Build fix
jmgasper e41826c
Performance changes for PM-2206
jmgasper 87a1eb2
Merge pull request #37 from topcoder-platform/PM-2206-develop
jmgasper 82232ba
Add in view to help with performance issue seen in dev
jmgasper c6d02f3
Automate migrations
kkartunov 6c0e4d8
use entrypoint
kkartunov 016a1b3
Merge pull request #40 from topcoder-platform/master
kkartunov b829e34
Merge pull request #41 from topcoder-platform/auto-migrations
kkartunov cb14e83
Prisma drift fix
jmgasper 38dd16c
Prisma drift fix
jmgasper cb285b3
Merge pull request #43 from topcoder-platform/migration-history-fix
jmgasper c0ff57f
Fix errors seen in build
jmgasper 170cbc5
Merge pull request #44 from topcoder-platform/build_fix
jmgasper 900e2da
Merge pull request #46 from topcoder-platform/submissionEndDateFix
jmgasper 9588596
Allow for registration phase reopening if submission or TG submission…
jmgasper 7ed9d4b
Merge pull request #48 from topcoder-platform/registrationReopen
jmgasper 0f5d076
Merge pull request #50 from topcoder-platform/master
jmgasper 0a4a03f
Merge pull request #51 from topcoder-platform/PM-2206-2nd-fix
jmgasper 6a9bb43
feat: added ai workflow id to default reviewer
hentrymartin c6c5463
fix: make scorecardId optional
hentrymartin 38c74bd
Merge pull request #53 from topcoder-platform/master
kkartunov 8c73a69
Merge branch 'develop' into pm-2540_3
hentrymartin d8c7eef
feat: added ai workflow id to default reviewer
hentrymartin 9a79208
Merge pull request #52 from topcoder-platform/pm-2540_3
hentrymartin 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 hidden or 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 hidden or 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 |
|---|---|---|
|
|
@@ -5,6 +5,7 @@ npm-debug.log* | |
| yarn-debug.log* | ||
| yarn-error.log* | ||
| .yarn | ||
| yarn.lock | ||
|
|
||
| # Runtime data | ||
| pids | ||
|
|
||
This file was deleted.
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.
[⚠️
performance]The removal of the
restore_cache_settings_for_buildandsave_cache_settingsanchors, along with their usage in thebuilddeploy_steps, could lead to increased build times if the node_modules directory is not cached elsewhere. Consider verifying if caching is handled in another part of the pipeline or if this removal is intentional and acceptable.