Skip to content
This repository was archived by the owner on Aug 22, 2025. It is now read-only.

ci: run lint workflow on pull request#1744

Merged
ChaituVR merged 4 commits into
masterfrom
run-lint-workflow-on-pr
Jun 8, 2025
Merged

ci: run lint workflow on pull request#1744
ChaituVR merged 4 commits into
masterfrom
run-lint-workflow-on-pr

Conversation

@wa0x6e

@wa0x6e wa0x6e commented Jun 8, 2025

Copy link
Copy Markdown
Contributor
  • Fix the build workflow title, which is mentioning lint, but is not linting
  • Run the lint workflow on pull request

@wa0x6e
wa0x6e requested a review from Copilot June 8, 2025 15:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Updates CI workflows to include lint on pull requests, cleans up build job naming, and reformats a multiline constructor call for readability.

  • Expand the lint workflow trigger to run on pull requests
  • Rename the build job and adjust its step to remove lint
  • Reformat Multicaller instantiation in the DeFi voting strategy to multiline

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/strategies/defi-app-voting/index.ts Reformatted Multicaller instantiation to a multiline object for clarity
.github/workflows/lint.yml Added pull_request to the on trigger for linting
.github/workflows/build.yml Renamed build_lint job to build and updated step name

@wa0x6e
wa0x6e requested a review from ChaituVR June 8, 2025 15:31

@ChaituVR ChaituVR left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

utAck

@wa0x6e
wa0x6e enabled auto-merge (squash) June 8, 2025 15:33
@wa0x6e

wa0x6e commented Jun 8, 2025

Copy link
Copy Markdown
Contributor Author

Screenshot 2025-06-08 at 19 35 13

This seems to be stuck for ever, since this workflow has been renamed

@ChaituVR
ChaituVR disabled auto-merge June 8, 2025 16:18
@ChaituVR
ChaituVR merged commit f48f823 into master Jun 8, 2025
10 checks passed
@ChaituVR
ChaituVR deleted the run-lint-workflow-on-pr branch June 8, 2025 16:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants