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

Issue / Rename Start Phases #201

Merged
merged 7 commits into from
Feb 5, 2025
Merged

Issue / Rename Start Phases #201

merged 7 commits into from
Feb 5, 2025

Conversation

dncsvr
Copy link
Member

@dncsvr dncsvr commented Feb 5, 2025 •
edited
Loading

Rename GetPhases as GetStartPhases in code, specs and docs for consistent
naming and use name Generate instead of Bake for generation mode

Tasks

  • rename GetPhases to GetStartPhases
  • rename context for start phases to StartContext
  • rename GetBakePhases to GetGeneratePhases
  • rename command arg to --generate
  • rename BakeContext to GeneratedContext
  • rename flag Bake to Generate
  • update documentation

Additional Tasks

  • add DomainModel parameter to ConfigureDomainServiceCollection helper
    • provide DomainModel to phase context in DomainLayer
  • use semicolon instead of curly braces for empty class declarations

dncsvr added 7 commits February 5, 2025 16:54
@dncsvr
init issue/rename-start-phases
f69059e
@dncsvr
replace Bake with Generate and use StartContext and StartPhases
e21dadc 
- rename `Bake` flag to `Generate`
- rename `GetPhases` to `GetStartPhases`
- rename `Context` for start mode to `StartContext`
- rename `BakeContext` to `GenerateContext`
- rename `GetBakePhases` to `GetGeneratePhases`
@dncsvr
update documentation
e7d2065
@dncsvr
add domain model parameter to ConfigureDomainServiceCollection helper
d0eddcb 
- remove unnecessary `UsingDomainModel`
@dncsvr
use semicolon instead of curly braces for empty class declarations
773d621
@dncsvr
fix unreleased format
906f5f6
@dncsvr
add Start mode info to introduced phases for layer docs
3830d79
@dncsvr dncsvr marked this pull request as ready for review February 5, 2025 15:11
@dncsvr dncsvr merged commit a605a91 into mouseless:main Feb 5, 2025
6 checks passed
@dncsvr dncsvr deleted the issue/rename-start-phases branch February 5, 2025 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cihandeniz cihandeniz cihandeniz 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.
2 participants
@dncsvr @cihandeniz