Skip to content

Enhancing Next.js: Updating Third-Party Scripts in Concurrent Fashion#54072

Closed
sanjaiyan-dev wants to merge 2 commits into
vercel:canaryfrom
sanjaiyan-dev:sanjaiyan-thrid-party-script-parallel
Closed

Enhancing Next.js: Updating Third-Party Scripts in Concurrent Fashion#54072
sanjaiyan-dev wants to merge 2 commits into
vercel:canaryfrom
sanjaiyan-dev:sanjaiyan-thrid-party-script-parallel

Conversation

@sanjaiyan-dev

Copy link
Copy Markdown
Contributor

Running the update-scripts.js function concurrently, as advised by @leerob in this comment (#53617 (comment)), I have decided not to modify the core Next.js code. Instead, I will be updating the scripts from third-party sources. I apologize if I have made any mistakes, and I am sorry if I have repeated the same mistakes again :)

@ijjk

ijjk commented Aug 15, 2023

Copy link
Copy Markdown
Member

Allow CI Workflow Run

  • approve CI run for commit: 6729cf4

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@leerob

leerob commented Aug 16, 2023

Copy link
Copy Markdown
Contributor

Could you please keep changes scoped within the examples/ folder? Thank you for understanding! 🙏

@leerob leerob closed this Aug 16, 2023
@sanjaiyan-dev

Copy link
Copy Markdown
Contributor Author

Could you please keep changes scoped within the examples/ folder? Thank you for understanding! 🙏

Yh sure ,and extremely sorry @leerob

@sanjaiyan-dev sanjaiyan-dev deleted the sanjaiyan-thrid-party-script-parallel branch August 21, 2023 07:21
@github-actions github-actions Bot added the locked label Sep 4, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants