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.

Sign up for GitHub

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

Jump to bottom

Create subfolder and updated path accordinly #393

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Create subfolder and updated path accordinly #393

merged 1 commit into from
Nov 18, 2024

Conversation

tech3371
Copy link
Contributor

Change Summary

closes #392

Overview

Group lambda code into its functional group. But I didn't add stack changes for lambda to only include its code. This seems more involve than planned.

Updated Files

  • all updated folder
    • All updated files only updates paths of new folder structure

@tech3371 tech3371 self-assigned this Nov 11, 2024
@tech3371 tech3371 force-pushed the folder_restructure branch 2 times, most recently from 858e1a3 to 2e1b86e Compare November 13, 2024 17:19
maxinelasp
maxinelasp approved these changes Nov 18, 2024
View reviewed changes
Copy link
Contributor

@maxinelasp maxinelasp left a comment

Choose a reason for hiding this comment

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

LGTM - do we want to split up the pipeline lambdas more...? Maybe move dependency lambdas into their own folder? Thats one that I could see getting updated more often than the others.

@tech3371
Copy link
Contributor Author

LGTM - do we want to split up the pipeline lambdas more...? Maybe move dependency lambdas into their own folder? Thats one that I could see getting updated more often than the others.

@maxinelasp That will be done in upcoming PR when I work on incorporating SPICE dependency changes

@tech3371 tech3371 merged commit 9a008ce into IMAP-Science-Operations-Center:dev Nov 18, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maxinelasp maxinelasp maxinelasp approved these changes

@laspsandoval laspsandoval Awaiting requested review from laspsandoval

Assignees

@tech3371 tech3371

Labels
Infrastructure
Projects
IMAP
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Group lambda code into its functional group
2 participants
@tech3371 @maxinelasp