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

MVP Rendering Restructure #482

Merged
merged 4 commits into from
Sep 30, 2024
Merged

MVP Rendering Restructure #482

merged 4 commits into from
Sep 30, 2024

Conversation

sc-ivanlieckens
Copy link
Collaborator

Description / Motivation

This PR continues the recent restructure of the repo to match the new starterkit.

  • Collapsed all projects to 1 single project, erasing the outdated Helix structure
  • Upgraded usage of Centralized Packages to latest version
  • Code cleanup
  • Several fixes to local container support (most importantly dotnet watch)

How Has This Been Tested?

Manual Testing

Local machine, docker & non-docker

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have read the Contributing guide.
  • My code/comments/docs fully adhere to the Code of Conduct.
  • My change is a code change.
  • My change is a documentation change and there are NO other updates required.

+ Collapsed all projects into 1
+ Upgraded Centralized Packages
+ Full nullability sweep
+ File scoped namespaces
+ Fixed docker dotnet watch
+ Improved Development config exclusion
+ Fixed docker layoutservice connection
@sc-ivanlieckens sc-ivanlieckens added enhancement New feature or request MVP Site labels Sep 27, 2024
@sc-ivanlieckens sc-ivanlieckens self-assigned this Sep 27, 2024
@robearlam robearlam merged commit 848e0e2 into main Sep 30, 2024
3 checks passed
@robearlam robearlam deleted the new/mvp-rendering-restructure branch September 30, 2024 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request MVP Site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants