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

command: charmcraft init to use charmcraft.yaml only #1149

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

syu-w
Copy link
Contributor

@syu-w syu-w commented Jul 11, 2023

Use unified charmcraft.yaml by default, that included metadata.yaml, actions.yaml and config.yaml.

Also added profiles descriptions and updated URL.

CRAFT-1795
CRAFT-1824

@syu-w
Copy link
Contributor Author

syu-w commented Jul 11, 2023

@syu-w syu-w force-pushed the move-charmcraft-init branch 2 times, most recently from 7da3782 to a5cf958 Compare July 12, 2023 22:11
@syu-w syu-w force-pushed the move-charmcraft-init branch 3 times, most recently from 763a60d to f5c9904 Compare July 19, 2023 21:11
@syu-w syu-w force-pushed the move-charmcraft-init branch 3 times, most recently from 7e44ed3 to 938a513 Compare August 1, 2023 21:30
@syu-w syu-w requested a review from lengau August 1, 2023 21:33
@jnsgruk
Copy link
Member

jnsgruk commented Aug 2, 2023

I think we're better off just removing the *-legacy templates - I can't see a particularly compelling reason to keep them.

Copy link
Collaborator

@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.

As long as @jnsgruk and @sergiusens are on the same page as each other about removing the -legacy templates I'm happy with that. Let's check in stand-up today and then you can merge either this as-is or without the legacy directories.

Thanks for all this work!

@syu-w syu-w force-pushed the move-charmcraft-init branch 2 times, most recently from 4d70b58 to c367e4d Compare August 15, 2023 18:43
@syu-w syu-w requested a review from mr-cal August 15, 2023 19:13
@syu-w syu-w enabled auto-merge (squash) August 15, 2023 19:44
@lengau lengau disabled auto-merge August 15, 2023 22:11
Copy link
Contributor

@mr-cal mr-cal left a comment

Choose a reason for hiding this comment

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

👍

Use unified charmcraft.yaml by default, that included `metadata.yaml`,
`actions.yaml` and `config.yaml`.

Also added profiles descriptions and updated URL.
@syu-w syu-w merged commit ca4df02 into canonical:main Aug 16, 2023
25 checks passed
@syu-w syu-w deleted the move-charmcraft-init branch August 16, 2023 20:59
@lengau lengau mentioned this pull request Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants