Enable DND and stateless flag in CIs#114
Conversation
|
👋 @marcospassos
☝️ Lastly, the title for the commit will come from the pull request title. So please provide a descriptive title that summarizes the changes in 50 characters or less using the imperative mood. |
There was a problem hiding this comment.
Pull Request Overview
This PR changes the default behavior of the stateless and dnd CLI flags to automatically enable when running in a CI environment.
- Default
statelessnow falls back toci.isCIinstead offalse. - Default
dndnow falls back toci.isCIinstead offalse.
Comments suppressed due to low confidence (3)
src/infrastructure/application/cli/cli.ts:429
- Consider adding unit tests to verify that the
statelessflag defaults totruewhenci.isCIistrueand tofalseotherwise.
stateless: configuration.stateless ?? ci.isCI,
src/infrastructure/application/cli/cli.ts:434
- Add unit tests to cover the new default behavior of
dnd, ensuring it is enabled when running in CI and disabled otherwise.
dnd: configuration.dnd ?? ci.isCI,
src/infrastructure/application/cli/cli.ts:429
- Update the CLI help text or project documentation to indicate that
statelessanddndflags now default totruein CI environments.
stateless: configuration.stateless ?? ci.isCI,
commit: |
Summary
This PR enables DND and stateless in CIs.
Checklist