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

cli: SNAPCRAFT_BUILD_ENVIRONMENT isn't deprecated #2179

Merged
merged 1 commit into from Jul 13, 2018

Conversation

kyrofa
Copy link
Contributor

@kyrofa kyrofa commented Jul 12, 2018

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on bug reports?
  • If this is a new feature. Have you discussed the design on the forum?
  • Have you successfully run ./runtests.sh static?
  • Have you successfully run ./runtests.sh unit?

SNAPCRAFT_CONTAINER_BUILDS is deprecated, but we have a confusing error message if both are provided, saying that SNAPCRAFT_BUILD_ENVIRONMENT is the one that's deprecated. Fix this to say that SNAPCRAFT_CONTAINER_BUILDS is the one that's deprecated.

SNAPCRAFT_CONTAINER_BUILDS is deprecated, but we have a confusing error
message if both are provided, saying that SNAPCRAFT_BUILD_ENVIRONMENT is
the one that's deprecated. Fix this to say that
SNAPCRAFT_CONTAINER_BUILDS is the one that's deprecated.

Signed-off-by: Kyle Fazzari <kyrofa@ubuntu.com>
@codecov-io
Copy link

Codecov Report

Merging #2179 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2179      +/-   ##
==========================================
+ Coverage   91.22%   91.22%   +<.01%     
==========================================
  Files         202      202              
  Lines       12848    12848              
  Branches     1906     1906              
==========================================
+ Hits        11720    11721       +1     
  Misses        765      765              
+ Partials      363      362       -1
Impacted Files Coverage Δ
snapcraft/cli/env.py 93.54% <ø> (ø) ⬆️
snapcraft/internal/elf.py 83.17% <0%> (+0.32%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 55dbbc8...26cf7b9. Read the comment docs.

Copy link
Collaborator

@sergiusens sergiusens left a comment

Choose a reason for hiding this comment

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

oops :-/

@sergiusens sergiusens merged commit 1ff7550 into canonical:master Jul 13, 2018
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.

None yet

3 participants