Skip to content

Fix Repo URL, align file names, add test pipeline - #2

Merged
TiMESPLiNTER merged 3 commits into
mainfrom
fix/repo-urls
Mar 15, 2026
Merged

Fix Repo URL, align file names, add test pipeline#2
TiMESPLiNTER merged 3 commits into
mainfrom
fix/repo-urls

Conversation

@TiMESPLiNTER

@TiMESPLiNTER TiMESPLiNTER commented Mar 15, 2026

Copy link
Copy Markdown
Owner

Fixes #1

Copilot AI review requested due to automatic review settings March 15, 2026 06:31

Copilot AI left a comment

Copy link
Copy Markdown

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 repo/package metadata and refactors the ESLint plugin entrypoints/utilities while adding a GitHub Actions workflow intended for CI.

Changes:

  • Bump package versions to 0.1.2 and update repository/bugs/homepage URLs to timesplinter/ts-final.
  • Rename/realign ESLint plugin module entrypoints and extract shared utilities into finalPluginUtils.ts.
  • Add a GitHub Actions workflow for CI and update the release workflow checkout action version.

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/final-decorator/src/index.ts Minor comment tweak in decorator implementation.
packages/final-decorator/package.json Version bump + repository metadata updates.
packages/eslint-plugin-final/src/index.ts Update plugin entry import to new filename.
packages/eslint-plugin-final/src/finalPluginUtils.ts Add shared TypeScript/AST utility helpers for the plugin rules.
packages/eslint-plugin-final/src/finalPlugin.ts Update imports to renamed utilities module.
packages/eslint-plugin-final/package.json Version bump + repository metadata updates.
README.md Reorganize usage documentation headings/sections.
.github/workflows/test.yml Add CI workflow (currently build-only).
.github/workflows/release.yml Update checkout action major version.
Comments suppressed due to low confidence (1)

packages/eslint-plugin-final/src/finalPlugin.ts:7

  • Same issue as in src/index.ts: importing ./finalPluginUtils.js from TypeScript source while using moduleResolution: "Node" can break compilation because TS may not resolve .js specifiers to .ts files. Align the tsconfig module resolution (Node16/NodeNext/Bundler) or drop the .js extension in the source import specifier.

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

You can also share your feedback on Copilot code review. Take the survey.

Comment thread packages/final-decorator/package.json Outdated
Comment thread .github/workflows/test.yml
Comment thread packages/eslint-plugin-final/src/index.ts
@TiMESPLiNTER
TiMESPLiNTER merged commit 177da4e into main Mar 15, 2026
1 check passed
@TiMESPLiNTER
TiMESPLiNTER deleted the fix/repo-urls branch March 15, 2026 06:49
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.

Wong URLs in package.json

2 participants