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!: deprecate enable-manifest and manifest-image-information #4618

Merged

Conversation

syu-w
Copy link
Contributor

@syu-w syu-w commented Feb 27, 2024

--enable-manifest and --manifest-image-information no longer supported in core24.

@syu-w syu-w force-pushed the work/CRAFT-2476-deprecated-info-manifest branch from fde93fd to d894ba2 Compare February 27, 2024 22:25
Copy link

codecov bot commented Feb 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.14%. Comparing base (6ff3e0b) to head (4493221).
Report is 31 commits behind head on feature/craft-application.

Additional details and impacted files
@@                      Coverage Diff                      @@
##           feature/craft-application    #4618      +/-   ##
=============================================================
+ Coverage                      88.83%   89.14%   +0.31%     
=============================================================
  Files                            327      331       +4     
  Lines                          22033    22080      +47     
=============================================================
+ Hits                           19573    19684     +111     
+ Misses                          2460     2396      -64     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

`--enable-manifest` and `--manifest-image-information` no longer
supported in core24.
@syu-w syu-w force-pushed the work/CRAFT-2476-deprecated-info-manifest branch from d894ba2 to 301a6fe Compare February 28, 2024 17:07
@syu-w syu-w changed the base branch from main to feature/craft-application February 28, 2024 17:07
@syu-w syu-w requested review from mr-cal and cmatsuoka February 28, 2024 18:03
Copy link
Collaborator

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

Will you please update https://snapcraft.io/docs/build-options to match this? (and mention the env vars to use instead)

@syu-w syu-w requested a review from sergiusens March 1, 2024 14:53
@syu-w
Copy link
Contributor Author

syu-w commented Mar 1, 2024

Will you please update https://snapcraft.io/docs/build-options to match this? (and mention the env vars to use instead)

Updated at https://forum.snapcraft.io/t/build-options/14250

@sergiusens sergiusens changed the title fix: deprecate enable-manifest and manifest-image-information feat!: deprecate enable-manifest and manifest-image-information Mar 4, 2024
@sergiusens sergiusens merged commit e426d07 into feature/craft-application Mar 4, 2024
8 of 11 checks passed
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.

3 participants