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

Project update #78

Merged
merged 4 commits into from Jun 7, 2021
Merged

Project update #78

merged 4 commits into from Jun 7, 2021

Conversation

yorodm
Copy link
Contributor

@yorodm yorodm commented Jun 1, 2021

  1. Remove deprecated packages and find a suitable replacement.
  2. Upgrade some versions.
  3. Some code cleanup

@yorodm yorodm requested a review from burke as a code owner June 1, 2021 20:34
@ghost ghost added the cla-needed label Jun 1, 2021
@burke
Copy link
Member

burke commented Jun 1, 2021

Fantastic, thanks! I think that CLA-bot sometimes needs another commit pushed after you click the buttons. Could you remove the commented code at the top of trust.rs?

@ghost ghost removed the cla-needed label Jun 1, 2021
@burke burke merged commit b4c8979 into Shopify:master Jun 7, 2021
@lugray lugray mentioned this pull request Sep 2, 2021
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

2 participants