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

Revamping GitHub Issue workflow #1656

Merged
merged 8 commits into from
Jul 25, 2019
Merged

Conversation

apurva-koti
Copy link
Collaborator

What changes are proposed in this pull request?

This pull request addresses the due improvements to the issue workflow we have been discussing. It adds a few strict issue templates and adds an issue policy.

How is this patch tested?

N/A

Release Notes

We have updated our GitHub issue policy. Please refer to the policy before filing an issue.

Is this a user-facing change?

  • No. You can skip the rest of this section.
  • Yes. Give a description of this change to be included in the release notes for MLflow users.

Our GitHub issue policy has been updated to streamline the review process and ensure issues are addressed appropriately.

What component(s) does this PR affect?

  • UI
  • CLI
  • API
  • REST-API
  • Examples
  • Docs
  • Tracking
  • Projects
  • Artifacts
  • Models
  • Scoring
  • Serving
  • R
  • Java
  • Python

How should the PR be classified in the release notes? Choose one:

  • rn/breaking-change - The PR will be mentioned in the "Breaking Changes" section
  • rn/none - No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" section
  • rn/feature - A new user-facing feature worth mentioning in the release notes
  • rn/bug-fix - A user-facing bug fix worth mentioning in the release notes
  • rn/documentation - A user-facing documentation change worth mentioning in the release notes

@apurva-koti apurva-koti added the enhancement New feature or request label Jul 24, 2019
README.rst Outdated Show resolved Hide resolved
ISSUE_POLICY.md Outdated Show resolved Hide resolved
ISSUE_POLICY.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@smurching smurching left a comment

Choose a reason for hiding this comment

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

Awesome this is looking great! Had a couple of suggestions

ISSUE_POLICY.md Outdated Show resolved Hide resolved
@apurva-koti apurva-koti removed the request for review from dbczumar July 24, 2019 21:44
Co-Authored-By: Siddharth Murching <smurching@gmail.com>
Co-Authored-By: Siddharth Murching <smurching@gmail.com>
Copy link
Collaborator

@smurching smurching left a comment

Choose a reason for hiding this comment

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

Two small suggestions, otherwise LGTM!

Co-Authored-By: Siddharth Murching <smurching@gmail.com>
@smurching smurching added LGTM rn/none List under Small Changes in Changelogs. and removed enhancement New feature or request labels Jul 24, 2019
@smurching smurching merged commit c682873 into mlflow:master Jul 25, 2019
avflor pushed a commit to avflor/mlflow that referenced this pull request Aug 22, 2020
Revamp GitHub issue templates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LGTM rn/none List under Small Changes in Changelogs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants