Skip to content

Conversation

grandizzy
Copy link
Collaborator

Motivation

  • could also be *artifact.source == *name_or_path ? @DaniPopes

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@0xferrous
Copy link
Contributor

isnt it better to run clippy and fmt checks on a pinned version that is updated periodically instead of waking up to failing CIs with no source changes?

@DaniPopes
Copy link
Member

DaniPopes commented Aug 20, 2025

We don't mind dealing with this that much, and also it doesn't happen that often

@DaniPopes DaniPopes enabled auto-merge (squash) August 20, 2025 06:37
@DaniPopes DaniPopes merged commit f887514 into foundry-rs:master Aug 20, 2025
22 checks passed
@github-project-automation github-project-automation bot moved this to Done in Foundry Aug 20, 2025
@grandizzy grandizzy deleted the fix-clippy-re-re-re branch August 20, 2025 07:16
grandizzy added a commit to grandizzy/foundry that referenced this pull request Aug 20, 2025
@grandizzy grandizzy mentioned this pull request Aug 20, 2025
3 tasks
grandizzy added a commit that referenced this pull request Aug 20, 2025
* fix(lint): explicit message when lint on build failures (#11224)

* fix(lint): lint only files that we build (#11247)

* fix: force 4844 txtype in blobhashes setter (#11355)

* test: add blobhashes repro

* fix: force 4844 tx type

* fix(forge): handle error if etherscan identifier cannot resolve config (#11356)

* fix(forge): handle error if etherscan identifier cannot resolve config

* warn on config failures

* fix: disable tx gas limit cap (#11347)

* fix(forge): write ordered deps in foundry.lock (#11360)

* chore: fix clippy (#11361)

* chore: bump version 1.3.2 (#11363)

* chore: fix cargo deny - update slab to 0.4.11

---------

Co-authored-by: Matthias Seitz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L-ignore Log: ignore PR in changelog
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants