Skip to content
This repository has been archived by the owner on Sep 6, 2022. It is now read-only.

⚡ Add support for copying of DayuTimeCode objects #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Eiken
Copy link
Member

@Eiken Eiken commented Jun 14, 2020

Moved dayu_timecode to gbk and created a pull request for our repo if dayu maintainers dont merge the PR

@Eiken Eiken requested a review from a team June 14, 2020 16:44
Copy link
Member

@simonrainerson simonrainerson left a comment

Choose a reason for hiding this comment

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

Commit message and PR title should say Add instead of Added. And your CI is having troubles, But code looks good to me.

@Eiken
Copy link
Member Author

Eiken commented Jun 14, 2020 via email

@Eiken Eiken force-pushed the erik/copy_support branch from d631e71 to d32ac2a Compare June 15, 2020 11:35
@Eiken Eiken changed the title ⚡ Added support for copying of DayuTimeCode objects ⚡ Add support for copying of DayuTimeCode objects Jun 15, 2020
@Eiken
Copy link
Member Author

Eiken commented Jun 15, 2020

Commit message and PR title should say Add instead of Added. And your CI is having troubles, But code looks good to me.

Changed to Add

@Eiken Eiken force-pushed the erik/copy_support branch from d32ac2a to 257cfe7 Compare June 15, 2020 13:42
@Eiken Eiken force-pushed the erik/copy_support branch from 257cfe7 to cea6c3e Compare June 15, 2020 13:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants