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

Comments

ci: run lint workflow on pull request#1744

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

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

Conversation

@wa0x6e
Copy link
Contributor

@wa0x6e wa0x6e commented Jun 8, 2025

  • 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
Copy link
Contributor

Copilot AI left a comment

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
Copy link
Member

@ChaituVR ChaituVR left a comment

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
Copy link
Contributor Author

wa0x6e commented Jun 8, 2025

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.

2 participants