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

Fix #2086 copyFiles in subPackage does not honour targetPath. #2087

Merged

Conversation

veelo
Copy link
Contributor

@veelo veelo commented Jan 22, 2021

No description provided.

@dlang-bot
Copy link
Collaborator

Thanks for your pull request and interest in making D better, @veelo! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please verify that your PR follows this checklist:

  • My PR is fully covered with tests (you can see the coverage diff by visiting the details link of the codecov check)
  • My PR is as minimal as possible (smaller, focused PRs are easier to review than big ones)
  • I have provided a detailed rationale explaining my changes
  • New or modified functions have Ddoc comments (with Params: and Returns:)

Please see CONTRIBUTING.md for more information.


If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment.

@veelo veelo force-pushed the issue2086-copyFiles-subPackage-targetPath branch from 3a5a40d to 3c03462 Compare January 22, 2021 16:45
@veelo veelo changed the title WIP #2086 copyFiles in subPackage does not honour targetPath. Fix #2086 copyFiles in subPackage does not honour targetPath. Jan 22, 2021
@veelo veelo force-pushed the issue2086-copyFiles-subPackage-targetPath branch 4 times, most recently from f0d219d to e96f614 Compare January 23, 2021 23:27
@veelo veelo force-pushed the issue2086-copyFiles-subPackage-targetPath branch from e96f614 to d9ef71b Compare January 23, 2021 23:40
@maxhaton maxhaton merged commit fb2b2b8 into dlang:master Feb 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants