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

Update DG & PR template to improve release process #2361

Merged
merged 2 commits into from
Sep 9, 2023

Conversation

tlylt
Copy link
Contributor

@tlylt tlylt commented Aug 27, 2023

What is the purpose of this pull request?

  • Documentation update
  • Bug fix
  • Feature addition or enhancement
  • Code maintenance
  • DevOps
  • Improve developer experience
  • Others, please explain:

Overview of changes:

  • adjust PR template to add a reviewer checklist
    • ensure version label is added
    • prepare release note
  • adjust DG
    • reminders for the PR template changes
    • remind to double check version

Anything you'd like to highlight/discuss:
nil

Testing instructions:
https://deploy-preview-2361--markbind-master.netlify.app/devguide/projectmanagement

Proposed commit message: (wrap lines at 72 characters)
Update docs & PR template to improve release process


Checklist: ☑️

  • Updated the documentation for feature additions and enhancements
  • Added tests for bug fixes or features
  • Linked all related issues
  • No unrelated changes

@codecov
Copy link

codecov bot commented Aug 27, 2023

Codecov Report

Merging #2361 (179eed1) into master (ebe9b9b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2361   +/-   ##
=======================================
  Coverage   45.11%   45.11%           
=======================================
  Files         123      123           
  Lines        5165     5165           
  Branches     1086     1086           
=======================================
  Hits         2330     2330           
  Misses       2516     2516           
  Partials      319      319           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@ong6 ong6 left a comment

Choose a reason for hiding this comment

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

Lgtm

@tlylt tlylt merged commit 102cad1 into MarkBind:master Sep 9, 2023
7 checks passed
@tlylt tlylt deleted the warn-version branch September 9, 2023 10:50
@tlylt tlylt added this to the v5.0.3 milestone Sep 9, 2023
@tlylt tlylt mentioned this pull request Nov 21, 2023
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants