Skip to content
This repository has been archived by the owner on Mar 28, 2023. It is now read-only.

Reapply "Minor cleanups" #1434

Merged
merged 2 commits into from
Feb 25, 2019
Merged

Conversation

placer14
Copy link
Member

@placer14 placer14 commented Feb 5, 2019

Reverts #1433 which reapplies #1306. This will be held until after #1425 is completed.

@coveralls
Copy link

coveralls commented Feb 5, 2019

Coverage Status

Coverage remained the same at 34.87% when pulling 0840109 on revert-1433-revert-1306-minor-cleanups into 4f30cff on master.

@gubatron
Copy link
Contributor

gubatron commented Feb 8, 2019

hi there folks, got a bit lost with this huge PR of mine.

If I understand correctly it was merged, but then removed due to issues with some other IPFS merge.

Let me know if you want me to try and rebase again and solve conflicts with IPFS stuff.

I was just shooting to reduce the huge number of warnings I was getting, as warnings usually lead to a bunch of sketchy issues, and also I was getting familiar with the codebase to see if I could be of more substantial help in my free time.

@cpacia
Copy link
Member

cpacia commented Feb 25, 2019

Merge conflicts :/

@cpacia cpacia force-pushed the revert-1433-revert-1306-minor-cleanups branch from b10a05e to f662c7a Compare February 25, 2019 17:59
@cpacia cpacia force-pushed the revert-1433-revert-1306-minor-cleanups branch from 26170ab to 0840109 Compare February 25, 2019 19:01
@cpacia cpacia merged commit 373e3a7 into master Feb 25, 2019
@gubatron
Copy link
Contributor

Cheers and thanks for that big merge!

@placer14 placer14 deleted the revert-1433-revert-1306-minor-cleanups branch March 21, 2019 22:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants