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

Migration of CopyFilesV2 task to Node 10 #13966

Merged
merged 8 commits into from
Dec 7, 2020

Conversation

EzzhevNikita
Copy link
Contributor

@EzzhevNikita EzzhevNikita commented Nov 24, 2020

Task name: CopyFilesV2
Description: Updated dependencies; Removed typings; Replaced deprecated MochaDone to Mocha.Done function; Fixed type errors in tests caused by changes in fs lib;

Documentation changes required: (Y/N) N

Added unit tests: (Y/N) Y

Attached related issue: (Y/N) #339

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

Hotfix branch

@EzzhevNikita EzzhevNikita marked this pull request as ready for review November 24, 2020 12:00
@EzzhevNikita EzzhevNikita requested a review from a team November 24, 2020 12:00
@anatolybolshakov anatolybolshakov requested a review from a team November 24, 2020 16:49
@EzzhevNikita EzzhevNikita merged commit e3b749b into master Dec 7, 2020
anatolybolshakov pushed a commit that referenced this pull request Jan 22, 2021
anatolybolshakov pushed a commit that referenced this pull request Jan 22, 2021
egor-bryzgalov pushed a commit that referenced this pull request Jan 25, 2021
* Revert "Migration of CopyFilesV2 task to Node 10 (#13966)"

This reverts commit e3b749b.

* Bumped task version

Co-authored-by: Mark Roghelia <[email protected]>
egor-bryzgalov pushed a commit that referenced this pull request Jan 25, 2021
* Revert "Migration of CopyFilesV2 task to Node 10 (#13966)"

This reverts commit e3b749b.

* Bumped task version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants