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

Other than code stuff #2

Open
wants to merge 33 commits into
base: main
Choose a base branch
from
Open

Other than code stuff #2

wants to merge 33 commits into from

Conversation

PrathamSahil
Copy link

No description provided.

@ph1ne4s
Copy link
Member

ph1ne4s commented Jun 21, 2023

Hi @PrathamSahil
Kindly ensure that you follow the same directory structure used in the project, which can be found at https://github.com/marsiitr/Restaurant-of-the-Future-3.0.
Also try to do it in a single PR.

@PrathamSahil
Copy link
Author

@ph1ne4s
We've made changes in the directory structure
Please review the changes and accept the pull request
Thanku

Nageshbansal
Nageshbansal requested changes Jul 18, 2023
View reviewed changes
Copy link
Member

@Nageshbansal Nageshbansal left a comment

Choose a reason for hiding this comment

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

CAD related files should be under the directory Mechanical Design

Nageshbansal
Nageshbansal requested changes Jul 18, 2023
View reviewed changes
Copy link
Member

@Nageshbansal Nageshbansal left a comment

Choose a reason for hiding this comment

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

rename the code directory with src

@ph1ne4s
Copy link
Member

ph1ne4s commented Nov 2, 2023

@PrathamSahil do the required changes asap.

@mohit-41
Copy link

mohit-41 commented Nov 2, 2023

@PrathamSahil do the required changes asap.

We have made the changes. Kindly verify.

@PrathamSahil
Copy link
Author

@ph1ne4s please review the changes and accept the pull request

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

@Nageshbansal Nageshbansal Nageshbansal requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@PrathamSahil @ph1ne4s @mohit-41 @Nageshbansal @priyanshumsr