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

[Sweep GHA Fix] The GitHub Actions run failed with... #214

Open
wants to merge 4 commits into
base: sweep/sweep_gha_fix_the_github_actions_run_fai_43
Choose a base branch
from

Conversation

sweep-ai[bot]
Copy link

@sweep-ai sweep-ai bot commented Jan 4, 2024

Description

This pull request includes the following changes:

  • Updated the base image in the Dockerfile to node:14.17.0
  • Updated the version of lerna in the Dockerfile to 4.1.0
  • Updated the build script in package.json to use tsc instead of lerna exec npm run build
  • Updated the devDependencies in package.json to use the latest versions of various packages
  • Updated the dependencies in package.json to use the latest versions of various packages
  • Updated the build script in packages/node-dev/package.json to include the -p flag
  • Updated the devDependencies in packages/node-dev/package.json to use the latest versions of various packages
  • Updated the buildFiles function in packages/node-dev/src/Build.ts to handle error types
  • Updated the tsconfig.json file in packages/node-dev to include the declarationMap and preserveConstEnums options
  • Updated the devDependencies in packages/workflow/package.json to use the latest versions of various packages
  • Updated the dependencies in packages/workflow/package.json to use the latest versions of various packages
  • Updated the Expression class in packages/workflow/src/Expression.ts to handle error types

Summary

  • Updated base image in Dockerfile to node:14.17.0
  • Updated lerna version in Dockerfile to 4.1.0
  • Updated build script in package.json to use tsc
  • Updated devDependencies and dependencies in package.json
  • Updated build script in packages/node-dev/package.json
  • Updated devDependencies in packages/node-dev/package.json
  • Updated buildFiles function in packages/node-dev/src/Build.ts
  • Updated tsconfig.json in packages/node-dev
  • Updated devDependencies and dependencies in packages/workflow/package.json
  • Updated Expression class in packages/workflow/src/Expression.ts

Copy link
Author

sweep-ai bot commented Jan 4, 2024

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

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.

0 participants