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

Standardize Action #296

Closed
wants to merge 24 commits into from
Closed

Standardize Action #296

wants to merge 24 commits into from

Conversation

ncalteen
Copy link
Collaborator

@ncalteen ncalteen commented Feb 22, 2024

This PR updates the first-interaction action to follow a more standardized approach as referenced in the TypeScript template. Additionally, it removes the container component to reduce action duration.

  • Updated various configuration files (ESlint, MarkdownLint, Prettier, Jest, YAMLLint)
  • Added Dependabot for npm and GitHub Actions dependencies
  • Added check-dist workflow to validate transpiled TypeScript
  • Added continuous-delivery workflow to handle automatic releases of new action versions
  • Updated continuous-integration/ci workflow to include unit test reporting
  • Switched from a separate license workflow to bundled licenses with vercel/ncc
  • Added linting workflow
  • Removed binary dependencies copied from actions/toolkit
  • Updated CODEOWNERS to actions-oss-maintainers team
  • Added/updated various repo files (CODE_OF_CONDUCT, CONTRIBUTING, SECURITY)
  • Updated README with a more complete usage example
  • Added 100% test coverage :D
  • Added various package.json scripts to handle formatting, linting, etc.

Closes #233
Closes #118
Closes #57
Closes #41
Closes #32
Closes #28
Closes #25
Closes #31
Closes #11
Closes #8

@ncalteen ncalteen self-assigned this Feb 22, 2024
@ncalteen ncalteen requested a review from a team as a code owner February 22, 2024 21:08
dist/sourcemap-register.js Dismissed Show dismissed Hide dismissed
@ncalteen
Copy link
Collaborator Author

Note: CI is going to fail as this workflow does not exist on main, hence additional permissions will not be available

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