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

Add prepublish script before publishing to aragonPM #905

Merged
merged 3 commits into from
Jul 16, 2019
Merged

Conversation

sohkai
Copy link
Contributor

@sohkai sohkai commented Jul 5, 2019

Requires aragon/aragon-cli#571 (and the aragonCLI upgrade).

cc @0xGabi

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.732% when pulling a0ee042 on apm-prepublish into 4207ef0 on master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.732% when pulling a0ee042 on apm-prepublish into 4207ef0 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.732% when pulling a0ee042 on apm-prepublish into 4207ef0 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.732% when pulling a0ee042 on apm-prepublish into 4207ef0 on master.

@coveralls
Copy link

coveralls commented Jul 5, 2019

Coverage Status

Coverage decreased (-0.2%) to 98.259% when pulling 275d279 on apm-prepublish into 3126dc4 on master.

@0xGabi
Copy link
Contributor

0xGabi commented Jul 5, 2019

LGTM

We will probably release a major version for the aragonCLI cause we introduced a breaking change making ipfs optional.

We could also release a minor only with the PRs for updating apm publish.

@0xGabi
Copy link
Contributor

0xGabi commented Jul 8, 2019

aragon apm publish now needs confirmation so I included the --skip-confirmation flag to skip confirmation.

@sohkai What do you think about propagate content. Is this something we want to do as default on each publish? There is a new flag: --propagate-content that we might want to configure as well.

@sohkai
Copy link
Contributor Author

sohkai commented Jul 8, 2019

@0xGabi Having that flag would be useful too, although if it prompts users by default, I think that's fine as well.

@0xGabi
Copy link
Contributor

0xGabi commented Jul 9, 2019

Oh ok, I thought it might break the CI. But I just realize that all checks have pass 😅

Will revert last commit then.

@sohkai
Copy link
Contributor Author

sohkai commented Jul 9, 2019

Ah, we don't publish from the CI 😄

@0xGabi
Copy link
Contributor

0xGabi commented Jul 9, 2019

@sohkai Ready 👍

@sohkai sohkai merged commit 87d6177 into master Jul 16, 2019
@sohkai sohkai deleted the apm-prepublish branch July 16, 2019 14:21
MickdeGraaf pushed a commit to MickdeGraaf/aragon-apps that referenced this pull request Jan 28, 2020
ramilexe pushed a commit to ConsiderItDone/aragon-apps that referenced this pull request Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants