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

Feature/issue 47/initial setup #48

Merged
merged 12 commits into from
Oct 3, 2024
Merged

Feature/issue 47/initial setup #48

merged 12 commits into from
Oct 3, 2024

Conversation

maxgrove42
Copy link
Collaborator

@maxgrove42 maxgrove42 commented Oct 3, 2024 •
edited
Loading

Issue 47

Description

  • Initial setup of repository for Django
  • Adds linting capabilities to ensure code styling compatibility

Change Type (delete non-relevant options)

  • 💡 New feature (non-breaking change which adds functionality)

Dependencies

  • This branch contains a breaking change ⚠️
    • Requires developers to follow development setup instructions in README

@maxgrove42 maxgrove42 changed the title [DRAFT] Feature/issue 47/initial setup Feature/issue 47/initial setup Oct 3, 2024
TekuriSaiAkhil
TekuriSaiAkhil approved these changes Oct 3, 2024
View reviewed changes
Copy link
Collaborator

@TekuriSaiAkhil TekuriSaiAkhil 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 to me!!

@estchang
Copy link
Collaborator

estchang commented Oct 3, 2024

Thanks Max! Setup works great for me!

QahtanAlJammali
QahtanAlJammali approved these changes Oct 3, 2024
View reviewed changes
Copy link
Collaborator

@QahtanAlJammali QahtanAlJammali 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! Thank you Max.

arsalananwar11
arsalananwar11 approved these changes Oct 3, 2024
View reviewed changes
Copy link
Collaborator

@arsalananwar11 arsalananwar11 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM, Thanks Max!

@maxgrove42 maxgrove42 merged commit 64ac31f into develop Oct 3, 2024
2 checks passed
@maxgrove42 maxgrove42 deleted the feature/issue-47/initial-setup branch October 3, 2024 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arsalananwar11 arsalananwar11 arsalananwar11 approved these changes

@TekuriSaiAkhil TekuriSaiAkhil TekuriSaiAkhil approved these changes

@QahtanAlJammali QahtanAlJammali QahtanAlJammali approved these changes

@estchang estchang estchang approved these 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
@maxgrove42 @estchang @arsalananwar11 @TekuriSaiAkhil @QahtanAlJammali