Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[DRAFT] [TRI-768] Update job imports in Trigger API route#5

Open
sweep-ai-deprecated[bot] wants to merge 4 commits intomain from
sweep/update-job-imports
Open

[DRAFT] [TRI-768] Update job imports in Trigger API route #5
sweep-ai-deprecated[bot] wants to merge 4 commits intomain from
sweep/update-job-imports

Conversation

@sweep-ai-deprecated
Copy link

@sweep-ai-deprecated sweep-ai-deprecated bot commented Jul 26, 2023

Description

This PR addresses TRI-768 by updating the job imports in the Trigger API route files.

Summary of Changes

  • Created an index.ts file in the jobs directory.
  • Moved the import statements for the individual job files from the Trigger API route files to the index.ts file.
  • Updated the Trigger API route files to import the index.ts file instead of the individual job files.

These changes make the code more organized and scalable, as adding new jobs will only require updating the index.ts file in the jobs directory.

Please review and merge this PR. Thank you!

Fixes #1.

To checkout this PR branch, run the following command in your terminal:

git checkout sweep/update-job-imports

@sweep-ai-deprecated sweep-ai-deprecated bot added the sweep Assigns Sweep to an issue or pull request. label Jul 26, 2023
Copy link
Author

@sweep-ai-deprecated sweep-ai-deprecated bot left a comment

Choose a reason for hiding this comment

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

No changes required. The modifications in the import statements across the files are consistent and seem to be part of a refactoring effort to centralize the job imports into a single index file in each respective directory. This makes the code more maintainable and easier to understand.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

sweep Assigns Sweep to an issue or pull request.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Sweep(slow): [TRI-768] The CLI init command should generate a jobs/index file and jobs should be imported there.

0 participants

AltStyle によって変換されたページ (->オリジナル) /