Skip to content

Install a specific slither commit#3651

Merged
mergify[bot] merged 1 commit intodevelopfrom
slither-version
Oct 4, 2022
Merged

Install a specific slither commit#3651
mergify[bot] merged 1 commit intodevelopfrom
slither-version

Conversation

@maurelian
Copy link
Contributor

@maurelian maurelian commented Oct 4, 2022

Description

Updates the CI builder to install slither from a specific commit, which will allow us to run it on our code without having to remove the forge test files.

Tests

I tested the build locally.

@changeset-bot
Copy link

changeset-bot bot commented Oct 4, 2022

🦋 Changeset detected

Latest commit: 3bd0527

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/ci-builder Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added 2-reviewers A-pkg-contracts-bedrock Area: packages/contracts-bedrock A-ops Area: ops labels Oct 4, 2022
@maurelian maurelian marked this pull request as draft October 4, 2022 18:59
@github-actions github-actions bot removed the A-pkg-contracts-bedrock Area: packages/contracts-bedrock label Oct 4, 2022
@maurelian maurelian marked this pull request as ready for review October 4, 2022 19:36
@maurelian maurelian requested review from mslipper and zhwrd October 4, 2022 19:36
@maurelian maurelian requested a review from tynes October 4, 2022 20:22
Copy link
Contributor

@tynes tynes left a comment

Choose a reason for hiding this comment

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

Generally looks good to me, will need to trigger a release after merge

@mergify
Copy link
Contributor

mergify bot commented Oct 4, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify mergify bot merged commit b979d86 into develop Oct 4, 2022
@mergify mergify bot deleted the slither-version branch October 4, 2022 20:25
@mergify
Copy link
Contributor

mergify bot commented Oct 4, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot removed the on-merge-train label Oct 4, 2022
maurelian added a commit that referenced this pull request Oct 5, 2022
This was referenced Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ops Area: ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants