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

Correct GitHub Pull Request Template - Fixes #520 #521

Merged
merged 1 commit into from
Feb 4, 2019

Conversation

PlagueHO
Copy link
Member

@PlagueHO PlagueHO commented Feb 3, 2019

Pull Request (PR) description

This PR removes the reference to the deprecated BestPractices.MD document from the GitHub PR template.

Also corrects one of the minor markdown violations in it.

This Pull Request (PR) fixes the following issues

Task list

  • Added an entry under the Unreleased section of the change log in
    CHANGELOG.md. Entry should say what was changed, and how that affects
    users (if applicable).
  • Resource documentation added/updated in README.md.
  • Resource parameter descriptions added/updated in README.md, schema.mof
    and comment-based help.
  • Comment-based help added/updated.
  • Localization strings added/updated in all localization files as
    appropriate.
  • Examples appropriately added/updated.
  • Unit tests added/updated. See
    DSC Resource Testing Guidelines.
  • Integration tests added/updated (where possible). See
    DSC Resource Testing Guidelines.
  • New/changed code adheres to
    DSC Resource Style Guidelines
    and
    Best Practices.

This change is Reviewable

@PlagueHO PlagueHO requested a review from mhendric February 3, 2019 07:58
@PlagueHO PlagueHO added the needs review The pull request needs a code review. label Feb 3, 2019
@codecov-io
Copy link

codecov-io commented Feb 3, 2019

Codecov Report

Merging #521 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@         Coverage Diff         @@
##            dev   #521   +/-   ##
===================================
  Coverage    74%    74%           
===================================
  Files        27     27           
  Lines      4031   4031           
  Branches      4      4           
===================================
  Hits       3005   3005           
  Misses     1022   1022           
  Partials      4      4

Copy link
Contributor

@mhendric mhendric left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@mhendric
Copy link
Contributor

mhendric commented Feb 3, 2019

Looks good to me @PlagueHO . Just needs a rebase and we're good to merge.

@mhendric mhendric added waiting for author response The pull request is waiting for the author to respond to comments in the pull request. and removed needs review The pull request needs a code review. labels Feb 3, 2019
@PlagueHO
Copy link
Member Author

PlagueHO commented Feb 4, 2019

Should be good to go now @mhendric

Copy link
Contributor

@mhendric mhendric left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@mhendric mhendric merged commit ac0411a into dsccommunity:dev Feb 4, 2019
@PlagueHO PlagueHO deleted the Issue-520 branch February 4, 2019 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting for author response The pull request is waiting for the author to respond to comments in the pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pull Request Template Refers to BestPractices.MD and Should Not
3 participants