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): 7.0.2 #13895

Merged
merged 6 commits into from
Nov 6, 2023
Merged

chore(release): 7.0.2 #13895

merged 6 commits into from
Nov 6, 2023

Conversation

shlokamin
Copy link
Member

@shlokamin shlokamin commented Nov 1, 2023

Overview

This PR tracks changes in the 7.0.2 hotfix

@shlokamin shlokamin added the DO NOT MERGE Indicates a PR should not be merged, even if there's a shiny green merge button available label Nov 1, 2023
@shlokamin shlokamin changed the title fix(app): improve robot update via usb (#13849) chore(release): 7.0.2 Nov 1, 2023
Copy link

codecov bot commented Nov 1, 2023

Codecov Report

Merging #13895 (f6b79cb) into release (692888c) will increase coverage by 0.00%.
The diff coverage is 51.66%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           release   #13895   +/-   ##
========================================
  Coverage    71.23%   71.23%           
========================================
  Files         2432     2432           
  Lines        68424    68419    -5     
  Branches      8047     8046    -1     
========================================
+ Hits         48740    48741    +1     
  Misses       17797    17797           
+ Partials      1887     1881    -6     
Flag Coverage Δ
app 68.84% <51.66%> (+0.02%) ⬆️
g-code-testing 96.44% <ø> (ø)
notify-server 89.13% <ø> (ø)
shared-data 74.08% <ø> (ø)
usb-bridge 76.81% <ø> (ø)

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

Files Coverage Δ
...p-shell/src/protocol-analysis/executeAnalyzeCli.ts 20.00% <ø> (ø)
...pp/src/organisms/Devices/InstrumentsAndModules.tsx 64.15% <ø> (ø)
...tings/UpdateBuildroot/RobotUpdateProgressModal.tsx 70.88% <ø> (+0.88%) ⬆️
...obotSettings/UpdateBuildroot/useRobotUpdateInfo.ts 80.76% <100.00%> (ø)
...anisms/PipetteWizardFlows/getPipetteWizardSteps.ts 86.66% <100.00%> (+15.23%) ⬆️
...tteWizardFlows/getPipetteWizardStepsForProtocol.ts 100.00% <100.00%> (+30.00%) ⬆️
app/src/organisms/PipetteWizardFlows/index.tsx 1.63% <ø> (+0.02%) ⬆️
app/src/redux/robot-api/http.ts 86.36% <ø> (ø)
app/src/redux/robot-update/epic.ts 71.31% <100.00%> (+0.22%) ⬆️
app-shell/src/http.ts 18.00% <0.00%> (ø)
... and 9 more

shlokamin and others added 5 commits November 1, 2023 18:38
Closes RQA-1827

* fix(app): fix attachment flow firmware step filter

Fixes the logic for determining whether or not the firmware step is necessary during an attachment
flow. Because it's not possible to check whether an instrument or module needs a firmware update
until it is attached, do not filter out the step entirely. Instead, if the instrument is known to
have the latest firmware, gracefully proceed to the next step.
This prevents python from picking up versions of dependencies installed
on the system, which will break offline analysis if the dependencies
installed globally are old.
… 22 (#13907)

Zip file robot updates ocassionally cause issues with retrying updates, especially on Windows.
Remove the retry logic and file upload progress until sorted.

Co-authored-by: Shlok Amin <[email protected]>
@shlokamin shlokamin marked this pull request as ready for review November 6, 2023 18:39
@shlokamin shlokamin requested review from a team as code owners November 6, 2023 18:39
@shlokamin shlokamin requested review from ncdiehl11 and sfoster1 and removed request for a team November 6, 2023 18:39
Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@shlokamin shlokamin merged commit f6b79cb into release Nov 6, 2023
106 of 112 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE Indicates a PR should not be merged, even if there's a shiny green merge button available
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants