Skip to content
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

Update definition for trunk-toolbox to include latest support for neveredit rule #859

Merged
merged 13 commits into from
Sep 3, 2024

Conversation

EliSchleifer
Copy link
Member

  1. trunk toolbox definition and support changes with version 0.4.1. toolbox can now run correctly with hold-the-line standard upstream support. Also adds support for a new rule "neveredit" which protect files from editing once checked in.
  2. backwards support for old version of toolbox is maintained.

Copy link

trunk-io bot commented Aug 30, 2024

⏱️ 2h 25m total CI duration on this PR
Job Cumulative Duration Recent Runs
Linter Tests macOS 59m 🟥🟩
Linter Tests ubuntu-latest 45m 🟥🟩
Tool Tests (ubuntu-latest) 16m 🟩🟩
Tool Tests (macOS) 8m 🟩🟩
CodeQL-Build 7m 🟩🟩🟩🟩
Trunk Check runner [linux] 4m 🟩🟩
Action Tests 3m 🟩🟩
Windows Linter Tests 2m 🟩🟩
Repo Tests / Plugin Tests 1m 🟩🟩
Detect changed files 14s 🟩🟩
Aggregate Test Results 5s 🟥🟥🟥

settingsfeedbackdocs ⋅ learn more about trunk.io

Copy link
Collaborator

@TylerJang27 TylerJang27 left a comment

Choose a reason for hiding this comment

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

One copy nit

lint:
definitions:
- name: trunk-toolbox
description: repository level linting tools
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
description: repository level linting tools
description: Repository-level linting tools

@TylerJang27
Copy link
Collaborator

You'll need to also run

PLUGINS_TEST_UPDATE_SNAPSHOTS=true npm test linters/trunk-toolbox
git add linters/trunk-toolbox
git commit -m 'add new snapshot'
git push

@EliSchleifer EliSchleifer enabled auto-merge (squash) September 3, 2024 18:40
@EliSchleifer EliSchleifer merged commit 2c48cf4 into main Sep 3, 2024
15 checks passed
@EliSchleifer EliSchleifer deleted the eli/update-toolbox branch September 3, 2024 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants