Skip to content

ci: Remove lefthook path to make it accessible on Windows #12929

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

Merged
merged 3 commits into from
Jan 30, 2025

Conversation

Mariana-na
Copy link
Contributor

Summary

Remove path on lefthook.yml to allow it to be accessible with windows

To test: change a typeScript file, save and run git commit command on Windows

Captura de ecrã 2025-01-29 165325

Captura de ecrã 2025-01-29 164929

Note to reviewer: Please test on Linux/mac to confirm it doesn't block them

Related Linear tickets, Github issues, and Community forum posts

None

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@Mariana-na Mariana-na requested a review from tomi January 29, 2025 17:00
@Mariana-na Mariana-na changed the title remove lefthook path to be accessible on windows ci: Remove lefthook path to make it accessible on Windows Jan 29, 2025
@n8n-assistant n8n-assistant bot added the n8n team Authored by the n8n team label Jan 29, 2025
Co-authored-by: कारतोफ्फेलस्क्रिप्ट™ <[email protected]>
netroy
netroy previously requested changes Jan 29, 2025
Co-authored-by: कारतोफ्फेलस्क्रिप्ट™ <[email protected]>
Copy link
Collaborator

@tomi tomi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works on Mac. Thank you for fixing 🫶

@tomi tomi requested a review from netroy January 30, 2025 09:39
@tomi tomi dismissed netroy’s stale review January 30, 2025 09:40

Comments addressed

@netroy netroy removed their request for review January 30, 2025 09:45
Copy link

cypress bot commented Jan 30, 2025

n8n    Run #9051

Run Properties:  status check failed Failed #9051  •  git commit 0052538261: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 MiloradFilipovic 🗃️ e2e/*
Project n8n
Branch Review ADO-2984-setup-template-credentials-list-fix
Run status status check failed Failed #9051
Run duration 04m 21s
Commit git commit 0052538261: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 MiloradFilipovic 🗃️ e2e/*
Committer Milorad FIlipović
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 5
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 365
View all changes introduced in this branch ↗︎

Tests for review

Failed  39-projects.cy.ts • 1 failed test

View Output Video

Test Artifacts
Projects > when starting from scratch > should move resources between projects Test Replay Screenshots Video
Failed  29-templates.cy.ts • 0 failed tests

View Output

Test Artifacts
Failed  16-form-trigger-node.cy.ts • 0 failed tests

View Output

Test Artifacts
Failed  32-node-io-filter.cy.ts • 0 failed tests

View Output

Test Artifacts
Failed  8-http-request-node.cy.ts • 0 failed tests

View Output

Test Artifacts

The first 5 failed specs are shown, see all 26 specs in Cypress Cloud.

Copy link
Contributor

✅ All Cypress E2E specs passed

@Mariana-na Mariana-na merged commit 6258f0c into master Jan 30, 2025
38 checks passed
@Mariana-na Mariana-na deleted the remove-path-on-lefthook branch January 30, 2025 09:57
@janober
Copy link
Member

janober commented Feb 6, 2025

Got released with [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team Released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants