Skip to content
This repository was archived by the owner on Oct 8, 2020. It is now read-only.

[API BREAKING] Rename Xgit.Core.* modules to Xgit.*. #257

Merged
merged 2 commits into from
Dec 15, 2019

Conversation

scouten
Copy link
Collaborator

@scouten scouten commented Dec 15, 2019

Changes in This Pull Request

Another piece of the simplification / refactoring in progress: I think we no longer need the .Core namespace for Xgit modules.

Checklist

  • This PR represents a single feature, fix, or change.
  • All applicable changes have been documented.
  • There is test coverage for all changes.
  • All cases where a literal value is returned use the cover macro to force code coverage. n/a
  • Any code ported from jgit maintains all existing copyright and license notices.
  • If new files are ported from jgit, the path to the corresponding file(s) is included in the header comment. n/a
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment. n/a

@scouten scouten self-assigned this Dec 15, 2019
@codecov
Copy link

codecov bot commented Dec 15, 2019

Codecov Report

Merging #257 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #257   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          29     29           
  Lines        1337   1337           
=====================================
  Hits         1337   1337
Impacted Files Coverage Δ
lib/xgit/content_source.ex 100% <ø> (ø)
lib/xgit/object_id.ex 100% <ø> (ø)
lib/xgit/file_content_source.ex 100% <ø> (ø)
lib/xgit/ref.ex 100% <ø> (ø)
lib/xgit/dir_cache.ex 100% <ø> (ø)
lib/xgit/repository/working_tree.ex 100% <ø> (ø) ⬆️
lib/xgit/repository/on_disk.ex 100% <ø> (ø) ⬆️
lib/xgit/file_path.ex 100% <ø> (ø)
lib/xgit/tree.ex 100% <ø> (ø)
lib/xgit/file_mode.ex 100% <ø> (ø)
... and 10 more

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 7ece975...57be61c. Read the comment docs.

@scouten scouten merged commit 7a8ec0d into master Dec 15, 2019
@scouten scouten deleted the remove-core-prefix branch December 15, 2019 01:35
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.

1 participant