Skip to content
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.

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

Cleanup and Convert to TypeScript #43

Merged
merged 4 commits into from
Feb 13, 2025
Merged

Cleanup and Convert to TypeScript #43

merged 4 commits into from
Feb 13, 2025

Conversation

ncalteen
Copy link
Contributor

This pull request includes various changes to configuration files and GitHub workflows to enhance project automation, linting, and continuous integration processes. The most important changes include the addition of new GitHub Actions workflows, updates to linter configurations, and modifications to environment and attribute files.

GitHub Actions Workflows:

Configuration Updates:

  • .env.example: Added example environment variables and inputs for testing with @github/local-action.
  • .github/codeql/codeql-config.yml: Configured CodeQL to ignore specific paths, such as node_modules and dist.
  • .github/dependabot.yml: Configured Dependabot to manage updates for GitHub Actions and npm dependencies, with separate groups for development and production dependencies.

Workflow Removals:

Action Updates

  • Converted to TypeScript
  • Added 100% test coverage
  • Adjusted inputs to all follow same format
  • Moved documentation to README.md
  • Updated to ESLint v9
  • Switched from @vercel/ncc to rollup.js for transpilation
  • Fixed several logic errors

These changes streamline the CI/CD processes, improve code quality checks, and ensure that dependencies are kept up-to-date automatically.

@ncalteen ncalteen requested a review from lindluni February 13, 2025 18:16
@ncalteen ncalteen self-assigned this Feb 13, 2025
@Copilot Copilot bot review requested due to automatic review settings February 13, 2025 18:16
@ncalteen ncalteen requested a review from jefeish as a code owner February 13, 2025 18:16

Choose a reason for hiding this comment

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

PR Overview

This pull request cleans up the repository by updating CI/CD workflows and converting components to TypeScript, while also streamlining the linting, build, and deployment processes. Key changes include:

  • New GitHub Actions workflows for checking the transpiled code in the dist/ directory, continuous integration, continuous delivery, and linting.
  • Updates to configuration files such as .env.example, .prettierrc.yml, .markdown-lint.yml, .github/codeql/codeql-config.yml, and Dependabot settings.
  • Removal of deprecated workflows no longer needed.

Changes

File Description
.github/workflows/check-dist.yml Adds a workflow to verify that dist/ contains the expected build
.github/workflows/continuous-integration.yml Introduces CI checks for formatting, linting, and tests
.github/workflows/continuous-delivery.yml Automates release creation upon PR merge or manual trigger
.github/workflows/linter.yml Sets up a dedicated workflow for linting using super-linter
.markdown-lint.yml Configures markdown linting rules
.prettierrc.yml Sets up Prettier formatting rules
.github/dependabot.yml Configures Dependabot for managing dependencies
.github/codeql/codeql-config.yml Configures CodeQL to ignore specific paths

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

Tip: Copilot only keeps its highest confidence comments to reduce noise and keep you focused. Learn more

@ncalteen ncalteen merged commit 620bbfb into main Feb 13, 2025
17 checks passed
@ncalteen ncalteen deleted the ncalteen/refactor branch February 18, 2025 18:55
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.

2 participants