New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: Lint on build #806

Merged

arcoraven merged 2 commits into main from ph/lintFixes

Dec 10, 2024

Contributor

arcoraven commented Dec 10, 2024 •

edited by pr-codex bot

Loading

PR-Codex overview

This PR focuses on refactoring code to replace parseInt with Number.parseInt, updating workflow files, and removing unused files.

Detailed summary

Renamed .github/workflows/npmPublish.yml to .github/workflows/build.yml.
Replaced parseInt with Number.parseInt in multiple files.
Added a new lint workflow in .github/workflows/lint.yml.
Deleted unused files, including .eslintrc.json and .npm.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

arcoraven added 2 commits

December 10, 2024 18:34


          chore: Biome linter fixes

46406dd


          chore: Add lint on build

80d4628

socket-security bot commented Dec 10, 2024

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: npm/@eslint-community/[email protected], npm/@eslint-community/[email protected], npm/@eslint/[email protected], npm/@eslint/[email protected], npm/@humanwhocodes/[email protected], npm/@humanwhocodes/[email protected], npm/@humanwhocodes/[email protected], npm/@humanwhocodes/[email protected], npm/@types/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/@typescript-eslint/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected]

View full report↗︎

arcoraven merged commit 04fdca0 into main

5 of 6 checks passed

arcoraven deleted the ph/lintFixes branch

December 10, 2024 11:02

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet