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

Initial Dev to Main codebase merge #27

Merged
merged 135 commits into from
Jul 12, 2023
Merged

Initial Dev to Main codebase merge #27

merged 135 commits into from
Jul 12, 2023

Conversation

YouruiR
Copy link
Contributor

@YouruiR YouruiR commented Jul 12, 2023

Overview

Initial merge from dev to main

  • Bug
  • Feature
  • Tech Debt

Description

Merging our codebase from dev to main for soft "launch" of NextInspect
Testing npm package CD deployment pipeline

mbildstein and others added 30 commits May 23, 2023 18:20
extension and package

Co-authored-by: davidan1989 <[email protected]>
Co-authored-by: William Nguyen <[email protected]>
Co-authored-by: Michael Bildstein <[email protected]>
## Overview

Issue Type

- [ ] Bug
- [ ] Feature
- [x] Tech Debt

Description
Added a yml file for the initial setup of GitHub actions


Signed-off-by: William Nguyen <[email protected]>
Signed-off-by: William Nguyen <[email protected]>
Signed-off-by: William Nguyen <[email protected]>
Signed-off-by: William Nguyen <[email protected]>
Signed-off-by: William Nguyen <[email protected]>
Signed-off-by: William Nguyen <[email protected]>
manifest file configured with minimum required settings
Added setup files and imports for express server with typings.

Added controller folder.
Created express server and types folder
Signed-off-by: William Nguyen <[email protected]>
Signed-off-by: William Nguyen <[email protected]>
Copy link
Contributor

@wnguye03 wnguye03 left a comment

Choose a reason for hiding this comment

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

Looks good, let's merge it; the only thing I would recommend is to make everyone co-author on this specific PR; otherwise, its great

Copy link
Contributor

@li-lucia li-lucia left a comment

Choose a reason for hiding this comment

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

Looks great!

@YouruiR YouruiR merged commit 91215a9 into main Jul 12, 2023
1 check passed
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.

5 participants