Skip to content

Conversation

@thesleepyniko
Copy link
Collaborator

title, only changes are a .yml file which should on either push (which shouldn't be possible on main) or on pull request lints with both yarn and tsc, tested locally with act on ghcr.io/catthehacker/ubuntu:act-latest

@thesleepyniko thesleepyniko requested a review from Copilot October 30, 2025 09:51
@vercel
Copy link

vercel bot commented Oct 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
aces Ready Ready Preview Comment Oct 30, 2025 9:53am

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

This PR introduces a GitHub Actions workflow for running TypeScript compilation checks and linting on push and pull request events.

Key Changes:

  • Adds a new GitHub Actions workflow file for TypeScript type checking
  • Configures the workflow to run on both push and pull request events
  • Sets up Node.js environment and runs linting and TypeScript compilation checks

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

copilot suggested changes, update v2 -> v4

Co-authored-by: Copilot <[email protected]>
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

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@hanaeatsplanes
Copy link
Collaborator

lgtm

Copy link
Collaborator

@hanaeatsplanes hanaeatsplanes left a comment

Choose a reason for hiding this comment

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

good girl

Copy link
Collaborator

@tailhaver tailhaver left a comment

Choose a reason for hiding this comment

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

oops i didnt see this until now lgtm

@thesleepyniko thesleepyniko merged commit 3cfa654 into main Oct 31, 2025
10 checks passed
@hanaeatsplanes hanaeatsplanes deleted the nikos-linting-workflow branch January 5, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants