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

build(actions): add check action#1

Merged
nipsysdev merged 4 commits into
mainfrom
build/add_check_action
Apr 25, 2025
Merged

build(actions): add check action#1
nipsysdev merged 4 commits into
mainfrom
build/add_check_action

Conversation

@nipsysdev
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings April 25, 2025 20:47
Copy link
Copy Markdown

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 updates tooling configurations and workflows to add a new check action and streamline ESLint configuration.

  • Tailwind configuration quotes adjusted.
  • Version numbers in mise.toml are downgraded.
  • ESLint configuration migrated from an mjs to a js file with adjusted imports, and new GitHub workflows for deploy and check actions are added.

Reviewed Changes

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

Show a summary per file
File Description
tailwind.config.mjs Updated content quoting style.
mise.toml Downgraded Node and PNPM versions.
eslint.config.mjs Removed in favor of the updated eslint.config.js configuration.
eslint.config.js Added ESLint configuration with a potential alias inconsistency.
.github/workflows/deploy.yml Updated syntax format for the action's "uses" key.
.github/workflows/check.yml New workflow file for linting and building on push and PR.
Files not reviewed (7)
  • package.json: Language not supported
  • src/pages/experience/[positionId].astro: Language not supported
  • src/pages/fr/experience.astro: Language not supported
  • src/pages/fr/experience/[positionId].astro: Language not supported
  • src/pages/fr/index.astro: Language not supported
  • src/pages/index.astro: Language not supported
  • src/styles/global.scss: Language not supported
Comments suppressed due to low confidence (1)

eslint.config.js:5

  • [nitpick] The 'typescript-eslint' package is imported with a different alias from the one used earlier in the file. Consider using a consistent alias to avoid potential confusion.
import tseslint from "typescript-eslint";

@nipsysdev nipsysdev merged commit 1f448f3 into main Apr 25, 2025
2 checks passed
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