-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Add GitHub Actions workflow for NodeJS with Webpack #2671
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: al7566 <215473224+al7566@users.noreply.github.com>
Co-authored-by: al7566 <215473224+al7566@users.noreply.github.com>
Co-authored-by: al7566 <215473224+al7566@users.noreply.github.com>
… URL Co-authored-by: al7566 <215473224+al7566@users.noreply.github.com>
Co-authored-by: al7566 <215473224+al7566@users.noreply.github.com>
Co-authored-by: al7566 <215473224+al7566@users.noreply.github.com>
|
@al7566 is attempting to deploy a commit to the Sim Team on Vercel. A member of the Team first needs to authorize it. |
Greptile Summary
PR Description Notes:
Important Files Changed
Confidence score: 0/5
Sequence DiagramsequenceDiagram
participant User
participant GitHub
participant Runner
participant Node18
participant Node20
participant Node22
User->>GitHub: "Push to main or create PR"
GitHub->>GitHub: "Trigger webpack.yml workflow"
par "Matrix build: Node 18.x"
GitHub->>Runner: "Start ubuntu-latest runner"
Runner->>Runner: "Checkout repository code"
Runner->>Node18: "Setup Node.js 18.x"
Runner->>Node18: "Run npm install"
Runner->>Node18: "Run npx webpack"
Node18->>Runner: "Build complete"
and "Matrix build: Node 20.x"
GitHub->>Runner: "Start ubuntu-latest runner"
Runner->>Runner: "Checkout repository code"
Runner->>Node20: "Setup Node.js 20.x"
Runner->>Node20: "Run npm install"
Runner->>Node20: "Run npx webpack"
Node20->>Runner: "Build complete"
and "Matrix build: Node 22.x"
GitHub->>Runner: "Start ubuntu-latest runner"
Runner->>Runner: "Checkout repository code"
Runner->>Node22: "Setup Node.js 22.x"
Runner->>Node22: "Run npm install"
Runner->>Node22: "Run npx webpack"
Node22->>Runner: "Build complete"
end
Runner->>GitHub: "All matrix builds complete"
GitHub->>User: "Workflow completed"
|
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.
Additional Comments (1)
1 file reviewed, 1 comment
…=20.9.0) Co-authored-by: al7566 <215473224+al7566@users.noreply.github.com>
Fix Node.js version in webpack workflow for Next.js 16.1 compatibility
…stem Automated key management system with "find, store, inject, forget" workflow
al7566
left a comment
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.
Review
|
paths:
Invalid - creates a parse error thatprevents your workflow from running.paths:
Validbranches: [ main, 'release/v[0-9].[0-9]' ] Invalid - creates a parse errorbranches: [ main, release/v[0-9].[0-9] ] Patterns to match branches and tags feature/your-branch feature/your-branch feature/mona/the/octocat releases/mona-the-octocat Matches the exact name of a branch or tag name. main releases/mona-the-octocat releases every/tag feature ver-10-feature v2.0 v2.9 v2.0.0 Pattern Description of matches Example matches server.rb page.jsx index.js js/index.js src/js/app.js docs/file.txt docs/mona/octocat.txt docs/mona/hello-world.md docs/a/markdown/file.md dir/docs/my-file.txt space/docs/plan/space.doc js/README.md my-src/code/js/app.js path/their-post.md db/migrate-v1.0.sql db/sept/migrate-v1.sql '!README.md' Using an exclamation mark (!) in front of a pattern negates it. When a file matches a pattern and also matches a negative pattern defined later in the file, the file will not be included. hello.md Does not match README.md docs/hello.md '!README.md' README* Patterns are checked sequentially. A pattern that negates a previous pattern will re-include file paths. hello.md README.md README.doc Make a contribution Still need help? × |
Summary
Brief description of what this PR does and why.
Fixes #(issue)
Type of Change
Testing
How has this been tested? What should reviewers focus on?
Checklist
Screenshots/Videos