-
Notifications
You must be signed in to change notification settings - Fork 9k
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
docs: Updates link to luxon documentation #12352
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
|
n8n Run #8465
Run Properties:
|
Project |
n8n
|
Branch Review |
datetime-node-fix-luxon-docs-link
|
Run status |
Failed #8465
|
Run duration | 04m 10s |
Commit |
7be84b7015: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 riascho 🗃️ e2e/*
|
Committer | Ria |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
1
|
Flaky |
1
|
Pending |
0
|
Skipped |
0
|
Passing |
377
|
View all changes introduced in this branch ↗︎ |
Tests for review
14-mapping.cy.ts • 1 failed test
Test | Artifacts | |
---|---|---|
Data mapping > maps expressions from json view |
Test Replay
Screenshots
Video
|
The first 5 failed specs are shown, see all 39 specs in Cypress Cloud.
e2e/48-subworkflow-inputs.cy.ts • 1 flaky test
Test | Artifacts | |
---|---|---|
Sub-workflow creation and typed usage > works with Fields input source into JSON input source |
Test Replay
Screenshots
Video
|
|
3 similar comments
|
|
|
Hey. So the n8n code is not working for render. Its saying soemthing about
the nodejs being outdated. How do we fix that? Please let me know in simple
terms. Newbie programmer here!
…On Mon, Dec 23, 2024, 10:51 PM github-actions[bot] ***@***.***> wrote:
|
|
Summary
Just a quick fix to update the correct link to our luxon documentation.
Related Linear tickets, Github issues, and Community forum posts
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)