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

Release 1.7.0 #380

Merged
merged 3 commits into from
Aug 22, 2023
Merged

Release 1.7.0 #380

merged 3 commits into from
Aug 22, 2023

Conversation

@codecov
Copy link

codecov bot commented May 25, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (226fe9a) 32.94% compared to head (ce0f5ba) 32.94%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #380   +/-   ##
=======================================
  Coverage   32.94%   32.94%           
=======================================
  Files          22       22           
  Lines        4013     4013           
=======================================
  Hits         1322     1322           
  Misses       2560     2560           
  Partials      131      131           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@hanzei hanzei left a comment

Choose a reason for hiding this comment

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

Waiting for #378

@hanzei hanzei added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels May 26, 2023
@hanzei hanzei removed the 2: Dev Review Requires review by a core committer label Jun 19, 2023
@hanzei hanzei added this to the v1.7.0 milestone Jun 19, 2023
@mickmister
Copy link
Member Author

@DHaussermann This release is ready for QA review

@mickmister
Copy link
Member Author

@DHaussermann Heads up that a customer has asked about this release in order to get #378 fixed on their system. Also, we discussed before about getting an RC of this on community. Wondering if this can play a factor in QA review here

@DHaussermann
Copy link

/update-branch

@DHaussermann
Copy link

DHaussermann commented Aug 7, 2023

@mickmister or @hanzei
Release testing failed because some bugs don't appear resolved. I then realized this branch has no commits added since the original creation. There are fixes in master we're missing.

/update-branch does not seem to work for me in this repo. Can one of you please pull master up?

@mickmister
Copy link
Member Author

@DHaussermann PR is now updated with master

@Kshitij-Katiyar
Copy link
Contributor

@DHaussermann PR is now updated with master

@mickmister @DHaussermann Are you working on the testing of this release or brightscout QA should start on testing the things included in this release?

@mickmister
Copy link
Member Author

@Kshitij-Katiyar Dylan is OOO until next week. Brightscout conducting QA review sounds good to me 👍

@mickmister
Copy link
Member Author

@Kshitij-Katiyar Please take a look at the test cases documented here for release testing the plugin https://github.com/mattermost/mattermost-test-management/tree/main/data/test-cases/plugins/gitlab

The plan is to use a spreadsheet to track verification of the test case passes. I can get you that soon, but please check out the test cases in the meantime. Thanks

@mickmister
Copy link
Member Author

@@Kshitij-Katiyar Here are the documented test cases in Atlassian's UI https://mattermost.atlassian.net/projects/MM?selectedItem=com.atlassian.plugins.atlassian-connect-plugin:com.kanoah.test-manager__main-project-page#!/v2/testCases. You may not be able access, or maybe not make entries for test runs, which is why I suggested to use a spreadsheet.

Here are the same test cases in spreadsheet form
https://docs.google.com/spreadsheets/d/16u7HvW18Cd3pMQSSl5UBck5TOaJSvA8MQn46690RnwY/edit?usp=sharing`

Pasting the above here again as well. This is where the test cases are actually defined
https://github.com/mattermost/mattermost-test-management/tree/main/data/test-cases/plugins/gitlab

@mickmister
Copy link
Member Author

/update-branch

Copy link

@DHaussermann DHaussermann left a comment

Choose a reason for hiding this comment

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

Tested and passed

  • Reviewed PRs since last version bump
  • Ran through test cycle for GitLab
  • No new issues found
  • Version correctly bumped to 1.7.0
    LGTM!

Test cases for code preview must still be added to Zephyr but, this does not block the release.

@DHaussermann DHaussermann added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Aug 22, 2023
@mickmister mickmister merged commit e98971b into master Aug 22, 2023
9 checks passed
@mickmister mickmister deleted the release-bump-1.7.0 branch August 22, 2023 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants