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

Various snap fixes #698

Merged
merged 4 commits into from
Nov 16, 2019
Merged

Various snap fixes #698

merged 4 commits into from
Nov 16, 2019

Conversation

RAOF
Copy link
Contributor

@RAOF RAOF commented Nov 16, 2019

This uses the new MEDIA_UPLOAD_DIRECTORY support to make media uploads work, cleans up some of the scripts, and sets the version based on git tags.

RAOF added 4 commits November 16, 2019 15:26
This lets us unconditionally run things under `set-environment` and let it
handle the case where the necessary configuration hasn't yet been set.
This will make the version something like `0.3.0-alpha-2-65-g29178aa` at the moment,
as that was the most recent tag, but the commit tagged as `0.4.0` will get *that* as
the version
This makes building the *second* time work; otherwise there might be
an existing cargo-web install, and cargo will refuse to overwrite it.
@codecov
Copy link

codecov bot commented Nov 16, 2019

Codecov Report

Merging #698 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #698   +/-   ##
=======================================
  Coverage   39.25%   39.25%           
=======================================
  Files          72       72           
  Lines        9498     9498           
  Branches     2265     2265           
=======================================
  Hits         3728     3728           
  Misses       4713     4713           
  Partials     1057     1057

@@ -1,6 +1,6 @@
name: plume
base: core18
version: '0.3.0' # just for humans, typically '1.2+git' or '1.3.2'
adopt-info: plume
Copy link
Contributor

@igalic igalic Nov 16, 2019

Choose a reason for hiding this comment

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

does Snapcraft build nightlies?

@elegaanz elegaanz merged commit df6f398 into master Nov 16, 2019
@elegaanz elegaanz deleted the various-snap-fixes branch November 16, 2019 12:19
@RAOF
Copy link
Contributor Author

RAOF commented Nov 16, 2019 via email

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