+
Skip to content
This repository was archived by the owner on Jun 27, 2023. It is now read-only.

Conversation

loganwright
Copy link
Member

@loganwright loganwright commented Mar 21, 2017

It's always needed, people regularly ask, it's proven itself pretty consistent, time to go in core 🙌

@codecov-io
Copy link

codecov-io commented Mar 21, 2017

Codecov Report

Merging #37 into master will decrease coverage by 0.66%.
The diff coverage is 68.42%.

@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
- Coverage   93.58%   92.92%   -0.67%     
==========================================
  Files          29       30       +1     
  Lines         702      721      +19     
==========================================
+ Hits          657      670      +13     
- Misses         45       51       +6
Impacted Files Coverage Δ
Tests/CoreTests/UtilityTests.swift 92.85% <100%> (+1.94%) ⬆️
Sources/Core/WorkingDirectory.swift 53.84% <53.84%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed5c97a...97aaaec. Read the comment docs.

if file.contains(".build") {
directory = file.components(separatedBy: "/.build").first
} else {
directory = file.components(separatedBy: "/Sources").first
Copy link
Member

Choose a reason for hiding this comment

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

We should also have one for Packages. This is used in 3.1 when you do swift package edit

@loganwright loganwright merged commit f71ea20 into master Mar 21, 2017
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.

3 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载