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

Remove move_loader and librapay #11184

Merged
merged 4 commits into from
Jul 23, 2020
Merged

Remove move_loader and librapay #11184

merged 4 commits into from
Jul 23, 2020

Conversation

garious
Copy link
Contributor

@garious garious commented Jul 23, 2020

Problem

The Move build introduces dependency conflicts.

Summary of Changes

Remove it. If needed, it can be resurrected in the SPL repo.

@jackcmay
Copy link
Contributor

Should we also remove mention of Libra Move from the doc's implemented-proposals?

@jackcmay
Copy link
Contributor

Also need to remove sdk/src/move_loader.rs and related artifacts

@garious
Copy link
Contributor Author

garious commented Jul 23, 2020

@jackcmay, looks like CI is about to pass. This look good to you?

@codecov
Copy link

codecov bot commented Jul 23, 2020

Codecov Report

Merging #11184 into master will decrease coverage by 0.0%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #11184     +/-   ##
=========================================
- Coverage    82.1%    82.1%   -0.1%     
=========================================
  Files         319      318      -1     
  Lines       74007    74002      -5     
=========================================
- Hits        60810    60801      -9     
- Misses      13197    13201      +4     

@garious
Copy link
Contributor Author

garious commented Jul 23, 2020

post-merge reviews welcome

@garious garious merged commit 8b1b392 into solana-labs:master Jul 23, 2020
@CriesofCarrots
Copy link
Contributor

I'm excited about the 30min this cut off stable CI

@mvines mvines mentioned this pull request Jul 24, 2020
mvines pushed a commit to mvines/solana that referenced this pull request Aug 1, 2020
* Remove move_loader and librapay

* Remove Embedding Move from implemented proposals

* Remove Move variant from CI

* Remove move_loader ID
mvines pushed a commit that referenced this pull request Aug 2, 2020
* Remove move_loader and librapay

* Remove Embedding Move from implemented proposals

* Remove Move variant from CI

* Remove move_loader ID
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