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

feat: add init service and init command #522

Merged
merged 5 commits into from
Oct 22, 2024
Merged

Conversation

mr-cal
Copy link
Contributor

@mr-cal mr-cal commented Oct 11, 2024

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox?

Broken into a few commits:

  1. Imports the init command from its private location with only changes needed to satisfy the linters
  2. Adds an InitService, a bunch of tests, and fixes some bugs caught when adding tests
  3. Adds profile support

Due to the size, this may be easier to review the entire thing rather than per-commit.

Fixes #452
Fixes #508
Creates #530
(CRAFT-3542)

mr-cal and others added 2 commits October 17, 2024 11:56
Co-authored-by: Dariusz Duda <[email protected]>
Signed-off-by: Callahan Kovacs <[email protected]>
Signed-off-by: Callahan Kovacs <[email protected]>
@mr-cal mr-cal force-pushed the work/CRAFT-3542/init-command branch from 8a303f9 to f1c4cbc Compare October 17, 2024 16:56
@mr-cal mr-cal changed the title [draft] feat: add init service and init command feat: add init service and init command Oct 17, 2024
@mr-cal mr-cal marked this pull request as ready for review October 17, 2024 17:00
@mr-cal mr-cal force-pushed the work/CRAFT-3542/init-command branch 2 times, most recently from 4361548 to 7487921 Compare October 18, 2024 18:25
@mr-cal mr-cal requested review from tigarmo, dariuszd21 and lengau and removed request for dariuszd21 October 18, 2024 19:04
Signed-off-by: Callahan Kovacs <[email protected]>
@mr-cal mr-cal force-pushed the work/CRAFT-3542/init-command branch from 7487921 to 2e8007b Compare October 18, 2024 20:00
craft_application/commands/__init__.py Outdated Show resolved Hide resolved
craft_application/commands/other.py Show resolved Hide resolved
craft_application/util/file.py Outdated Show resolved Hide resolved
craft_application/services/init.py Outdated Show resolved Hide resolved
craft_application/services/init.py Outdated Show resolved Hide resolved
craft_application/commands/init.py Outdated Show resolved Hide resolved
docs/reference/changelog.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@tigarmo tigarmo left a comment

Choose a reason for hiding this comment

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

Looks good :) I'm approving assuming @lengau 's comments will be addressed

craft_application/commands/other.py Show resolved Hide resolved
Signed-off-by: Callahan Kovacs <[email protected]>
@mr-cal mr-cal requested a review from lengau October 21, 2024 14:00
@mr-cal mr-cal added the squash label Oct 21, 2024
Copy link
Contributor

@lengau lengau left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

craft_application/commands/init.py Show resolved Hide resolved
@mr-cal
Copy link
Contributor Author

mr-cal commented Oct 21, 2024

Waiting on #530 to land first.

@mr-cal mr-cal merged commit f2a4bec into main Oct 22, 2024
9 of 10 checks passed
@mr-cal mr-cal deleted the work/CRAFT-3542/init-command branch October 22, 2024 19:04
@mr-cal mr-cal mentioned this pull request Oct 23, 2024
3 tasks
linostar pushed a commit to linostar/craft-application that referenced this pull request Dec 4, 2024
Signed-off-by: Callahan Kovacs <[email protected]>
Co-authored-by: Dariusz Duda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Import and add profile support for the init command create generic init implementation
3 participants