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: default all py versions to robot-stack #14496

Merged
merged 1 commit into from
Feb 14, 2024

Conversation

sfoster1
Copy link
Member

We used to have different default projects (which happen, it turns out, in the deploy environment) for sdists and wheels and stuff because sdists only existed for flex dev and we hadn't released it yet. Now we have, and we should always be using our robot-stack project unless ot_project is set to something else.

Keep the other defaults around but make them ir (for internal release) instead of ot3.

This should hopefully prevent the pypi uploads from posting internal-release sdists.

Should fix this: https://github.com/Opentrons/opentrons/actions/runs/7902877476/job/21569838106

We used to have different default projects (which happen, it turns out,
in the deploy environment) for sdists and wheels and stuff because
sdists only existed for flex dev and we hadn't released it yet. Now we
have, and we should always be using our robot-stack project unless
ot_project is set to something else.

Keep the other defaults around but make them ir (for internal release)
instead of ot3.

This should hopefully prevent the pypi uploads from posting
internal-release sdists.
@sfoster1 sfoster1 requested review from a team as code owners February 14, 2024 22:10
@sfoster1 sfoster1 changed the base branch from edge to chore_release-7.2.0 February 14, 2024 22:12
Copy link

codecov bot commented Feb 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (14cfb6e) 67.77% compared to head (86ca0eb) 67.79%.
Report is 1 commits behind head on chore_release-7.2.0.

Additional details and impacted files

Impacted file tree graph

@@                   Coverage Diff                   @@
##           chore_release-7.2.0   #14496      +/-   ##
=======================================================
+ Coverage                67.77%   67.79%   +0.02%     
=======================================================
  Files                     2517     2517              
  Lines                    72009    72060      +51     
  Branches                  9274     9274              
=======================================================
+ Hits                     48801    48851      +50     
- Misses                   20993    20994       +1     
  Partials                  2215     2215              
Flag Coverage Δ
app 64.64% <ø> (ø)
g-code-testing 92.43% <ø> (ø)
hardware 57.54% <ø> (+0.24%) ⬆️
hardware-testing ∅ <ø> (∅)
protocol-designer 37.93% <ø> (ø)
shared-data 75.24% <ø> (-0.02%) ⬇️
step-generation 86.90% <ø> (ø)
system-server 96.04% <ø> (ø)
update-server 50.56% <ø> (ø)
usb-bridge 76.94% <ø> (ø)

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

see 8 files with indirect coverage changes

Copy link
Contributor

@SyntaxColoring SyntaxColoring left a comment

Choose a reason for hiding this comment

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

Spicy!

@sfoster1 sfoster1 merged commit 9f5c9d0 into chore_release-7.2.0 Feb 14, 2024
69 of 71 checks passed
@sfoster1 sfoster1 deleted the dont-deploy-other-releases-to-pypi branch February 14, 2024 23:08
TamarZanzouri pushed a commit that referenced this pull request Feb 16, 2024
We used to have different default projects (which happen, it turns out,
in the deploy environment) for sdists and wheels and stuff because
sdists only existed for flex dev and we hadn't released it yet. Now we
have, and we should always be using our robot-stack project unless
ot_project is set to something else.

Keep the other defaults around but make them ir (for internal release)
instead of ot3.

This should hopefully prevent the pypi uploads from posting
internal-release sdists.
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.

None yet

2 participants