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

chore(release): 6.1.0 #11508

Merged
merged 5 commits into from
Sep 27, 2022
Merged

chore(release): 6.1.0 #11508

merged 5 commits into from
Sep 27, 2022

Conversation

shlokamin
Copy link
Member

@shlokamin shlokamin commented Sep 27, 2022

Overview

Final bump PR for the production release of v6.1.0

  • Bump package versions to 6.1.0
  • Generate technical change logs from 6.0.1 > 6.1.0
  • Add user-facing release notes

Review requests

  • All package versions are listed as 6.1.0
  • Technical changelogs show correct dates and versions
  • User-facing release notes look good
    • Links to any issues and pull requests are correct
    • No spelling mistakes
    • All features and fixes we want to highlight to users are included

Risk assessment

Bump PR. Low risk if process outlined in RELEASING.md is correctly followed

@codecov
Copy link

codecov bot commented Sep 27, 2022

Codecov Report

Merging #11508 (a881d9d) into release_6.1.0 (ce4d01b) will decrease coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@                Coverage Diff                @@
##           release_6.1.0   #11508      +/-   ##
=================================================
- Coverage          74.46%   74.39%   -0.07%     
=================================================
  Files               2033     2025       -8     
  Lines              56565    56344     -221     
  Branches            5534     5534              
=================================================
- Hits               42120    41917     -203     
+ Misses             13210    13192      -18     
  Partials            1235     1235              
Flag Coverage Δ
app 74.48% <ø> (ø)
components 53.26% <ø> (ø)
protocol-designer 45.81% <ø> (ø)
step-generation 88.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
notify-server/notify_server/logging.py
...server/otupdate/common/name_management/__init__.py
notify-server/notify_server/server/__init__.py
update-server/otupdate/common/config.py
update-server/otupdate/openembedded/__init__.py
update-server/otupdate/buildroot/__init__.py
notify-server/notify_server/settings.py
notify-server/notify_server/__init__.py
...otupdate/common/name_management/pretty_hostname.py 34.14% <0.00%> (+5.19%) ⬆️

@shlokamin shlokamin changed the base branch from release to release_6.1.0 September 27, 2022 14:31
@shlokamin shlokamin marked this pull request as ready for review September 27, 2022 14:35
@shlokamin shlokamin requested review from a team as code owners September 27, 2022 14:35
CHANGELOG.md Outdated
@@ -176,6 +103,11 @@ See [Conventional Commits](https://conventionalcommits.org) for commit guideline
* **robot-server:** Minimize selected columns to speed up `GET /runs` ([#11002](https://github.com/Opentrons/opentrons/issues/11002)) ([4000e5b](https://github.com/Opentrons/opentrons/commit/4000e5bf543ea56260e441e47e561e97103fe435))


### Reverts

* Revert "fix(api): Disallow uploading a protocol with non matching references (#11457)" (#11475) ([02d39a0](https://github.com/Opentrons/opentrons/commit/02d39a0a8774e528da7a7ffadaabe9f45805c33e)), closes [#11457](https://github.com/Opentrons/opentrons/issues/11457) [#11475](https://github.com/Opentrons/opentrons/issues/11475)
Copy link
Member

Choose a reason for hiding this comment

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

Need to remove

CHANGELOG.md Outdated
* **api:** be prepared for aspirate after pick-up-tip ([#10914](https://github.com/Opentrons/opentrons/issues/10914)) ([cf705bc](https://github.com/Opentrons/opentrons/commit/cf705bc8638b7e6d23272eb7b025512812131936))
* **api:** Disallow uploading a protocol with non matching references ([#11457](https://github.com/Opentrons/opentrons/issues/11457)) ([5f29ebe](https://github.com/Opentrons/opentrons/commit/5f29ebe649f364faf470222d5758dfdddc62ed1a))
Copy link
Member

Choose a reason for hiding this comment

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

Need to remove

Copy link
Contributor

@mcous mcous left a comment

Choose a reason for hiding this comment

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

A couple little notes to go along with Sanniti's change request

@@ -6,9 +6,9 @@ log][]. For a list of currently known issues, please see the [Opentrons issue tr

---

# OT-2 Software Changes in 6.1.0-beta.1
# OT-2 Software Changes in 6.1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Do heading levels of the previous release's section below (OT-2 Software Changes in 6.0.1) need to be dropped (i.e. # > ##, ## > ###, etc.)

Disregard if we're not doing this anymore and I missed that discussion

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @mcous for catching this and @sanni-t for implementing!

@@ -6,9 +6,9 @@ log][]. For a list of currently known issues, please see the [Opentrons issue tr

---

# Opentrons App Changes in 6.1.0-beta.1
# Opentrons App Changes in 6.1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment about previous releases, if applicable

Copy link
Member

@sanni-t sanni-t left a comment

Choose a reason for hiding this comment

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

Lgtm! Would like to get someone else's review too since I made changes as well.

shared-data/CHANGELOG.md Outdated Show resolved Hide resolved
Copy link

@nusrat813 nusrat813 left a comment

Choose a reason for hiding this comment

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

LGTM

@laviera laviera self-requested a review September 27, 2022 18:05
@shlokamin shlokamin merged commit f627f63 into release_6.1.0 Sep 27, 2022
@shlokamin shlokamin deleted the chore_bump-v6.1.0 branch December 29, 2022 19:16
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.

6 participants