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

ref(repository): Repository sync with the RR2 #24

Merged
merged 3 commits into from
Jan 27, 2021

Conversation

rustatian
Copy link
Collaborator

@rustatian rustatian commented Jan 27, 2021

Reason for This PR

Repository synchronization with the SpiralScout repositories format

Description of Changes

  1. Remove cmd folder.
  2. Add CI and codecov to the repository.
  3. Fix races in the tests.
  4. Properly use of errors.Op.
  5. Add PR template.
  6. Add dependabot which checks dependencies for the update on a daily basis.

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

  • All commits in this PR are signed (git commit -s).
  • The reason for this PR is clearly provided (issue no. or explanation).
  • The description of changes is clear and encompassing.
    - [ ] Any required documentation changes (code and docs) are included in this PR.
    - [ ] Any user-facing changes are mentioned in CHANGELOG.md.
  • All added/changed functionality is tested.

@rustatian rustatian added A-other Area: other C-enhancement Category: enhancement. Meaning improvements of current module, transport, etc.. labels Jan 27, 2021
@rustatian rustatian requested a review from wolfy-j January 27, 2021 09:36
@rustatian rustatian self-assigned this Jan 27, 2021
@codecov-io
Copy link

codecov-io commented Jan 27, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@cf82328). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #24   +/-   ##
=========================================
  Coverage          ?   64.20%           
=========================================
  Files             ?       17           
  Lines             ?      989           
  Branches          ?        0           
=========================================
  Hits              ?      635           
  Misses            ?      280           
  Partials          ?       74           

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 cf82328...52f9292. Read the comment docs.

@rustatian rustatian marked this pull request as ready for review January 27, 2021 09:50
@rustatian rustatian merged commit 6d5b74b into master Jan 27, 2021
@rustatian rustatian deleted the refactor/repository_unification branch January 27, 2021 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-other Area: other C-enhancement Category: enhancement. Meaning improvements of current module, transport, etc..
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants