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

Marketplace remove hash from manifest #864

Merged
merged 3 commits into from
Apr 19, 2019

Conversation

NicolasMahe
Copy link
Member

@NicolasMahe NicolasMahe commented Apr 9, 2019

Start to implements #861 for CLI and Service Marketplace:
only remove unnecessary hash for now

@NicolasMahe NicolasMahe force-pushed the ss/marketplace-remove-hash-manifest branch from 562729c to da21f67 Compare April 16, 2019 11:30
@NicolasMahe NicolasMahe changed the title Marketplace remove hash and hashVersion from manifest Marketplace remove hash from manifest Apr 16, 2019
@NicolasMahe NicolasMahe marked this pull request as ready for review April 16, 2019 11:31
@NicolasMahe NicolasMahe force-pushed the ss/marketplace-remove-hash-manifest branch from 3754dc1 to 1f3e041 Compare April 17, 2019 10:40
@NicolasMahe
Copy link
Member Author

@mesg-foundation/core I remove the dependency to the other PR. Ready to review.

@ilgooz
Copy link
Contributor

ilgooz commented Apr 17, 2019

@NicolasMahe I added a fix commit, it's better to squash this one while merging

@NicolasMahe NicolasMahe force-pushed the ss/marketplace-remove-hash-manifest branch from f9cdf00 to 198b711 Compare April 17, 2019 11:44
@NicolasMahe
Copy link
Member Author

@NicolasMahe I added a fix commit, it's better to squash this one while merging

Done

ilgooz
ilgooz previously approved these changes Apr 18, 2019
antho1404
antho1404 previously approved these changes Apr 18, 2019
@NicolasMahe NicolasMahe dismissed stale reviews from antho1404 and ilgooz via 17d62bb April 19, 2019 03:51
@NicolasMahe NicolasMahe merged commit e2bd6cd into dev Apr 19, 2019
@NicolasMahe NicolasMahe deleted the ss/marketplace-remove-hash-manifest branch April 19, 2019 03:52
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