Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Upgrade Cargo dependencies (#27) #135

Merged
merged 9 commits into from
Sep 8, 2020
Merged

Upgrade Cargo dependencies (#27) #135

merged 9 commits into from
Sep 8, 2020

Conversation

evdokimovs
Copy link
Contributor

@evdokimovs evdokimovs commented Sep 4, 2020 •
edited
Loading

Part of #27

Synopsis

cargo outdated output is big, so we need to upgrade outdated deps.

Checklist

  • Created PR:
    • In draft mode
    • Name contains Draft: prefix
    • Name contains issue reference
    • Has k:: labels applied
    • Has assignee
  • Documentation is updated (if required)
  • Tests are updated (if required)
  • Changes conform code style
  • CHANGELOG entry is added (if required)
  • FCM (final commit message) is posted
    • and approved
  • Review is completed and changes are approved
  • Before merge:
    • Milestone is set
    • PR's name and description are correct and up-to-date
    • Draft: prefix is removed
    • All temporary labels are removed

@evdokimovs evdokimovs added enhancement Improvement of existing features or bugfix k::refactor Refactoring, technical debt elimination and other improvements of existing code base labels Sep 4, 2020
@evdokimovs evdokimovs added this to the 0.2.0 milestone Sep 4, 2020
@evdokimovs evdokimovs self-assigned this Sep 4, 2020
@evdokimovs
Copy link
Contributor Author

evdokimovs commented Sep 4, 2020 •
edited
Loading

FCM

Upgrade Cargo dependencies (#135, #27)

Additionally:
- add 'turn.db.redis.user' configuration option for Medea

@evdokimovs evdokimovs changed the title Draft: Upgrade dependencies (#27) Upgrade dependencies (#27) Sep 4, 2020
@evdokimovs evdokimovs marked this pull request as ready for review September 4, 2020 12:49
@evdokimovs evdokimovs requested a review from alexlapa September 4, 2020 12:50
alexlapa
alexlapa suggested changes Sep 4, 2020
View reviewed changes
Copy link
Collaborator

@alexlapa alexlapa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@evdokimovs ,

mockall + async_trait as discussed.

@alexlapa alexlapa self-requested a review September 8, 2020 08:35
@alexlapa alexlapa requested a review from tyranron September 8, 2020 08:35
@tyranron tyranron added the k::config Related to application configuration label Sep 8, 2020
@evdokimovs evdokimovs changed the title Upgrade dependencies (#27) Upgrade Cargo dependencies (#27) Sep 8, 2020
@evdokimovs evdokimovs merged commit d2b1c4b into master Sep 8, 2020
@evdokimovs evdokimovs deleted the upgrade-deps branch September 8, 2020 09:48
@evdokimovs evdokimovs mentioned this pull request Sep 16, 2020
Merged
16 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alexlapa alexlapa alexlapa approved these changes

@tyranron tyranron tyranron approved these changes

Assignees

@evdokimovs evdokimovs

Labels
enhancement Improvement of existing features or bugfix k::config Related to application configuration k::refactor Refactoring, technical debt elimination and other improvements of existing code base
Projects
None yet
Milestone
0.2.0
Development

Successfully merging this pull request may close these issues.
3 participants
@evdokimovs @tyranron @alexlapa