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(build): fix push targets in api and s-d #11879

Merged
merged 1 commit into from
Dec 13, 2022
Merged

chore(build): fix push targets in api and s-d #11879

merged 1 commit into from
Dec 13, 2022

Conversation

sfoster1
Copy link
Member

These push targets in the makefiles in these projects were using the wheel and sdist filenames as target dependencies, but there was no longer a pattern target that matched them.

These push targets in the makefiles in these projects were using the
wheel and sdist filenames as target dependencies, but there was no
longer a pattern target that matched them.
@sfoster1 sfoster1 requested review from a team as code owners December 13, 2022 20:37
Copy link
Contributor

@Laura-Danielle Laura-Danielle left a comment

Choose a reason for hiding this comment

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

lgtm and makes sense

Copy link
Contributor

@TamarZanzouri TamarZanzouri left a comment

Choose a reason for hiding this comment

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

pushed to an ot-3 and it solved the problem! thank you for fixing this so quick!

@codecov
Copy link

codecov bot commented Dec 13, 2022

Codecov Report

Merging #11879 (5bf6308) into edge (8dce64a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             edge   #11879   +/-   ##
=======================================
  Coverage   74.35%   74.35%           
=======================================
  Files        2150     2150           
  Lines       59414    59414           
  Branches     6237     6237           
=======================================
  Hits        44176    44176           
  Misses      13768    13768           
  Partials     1470     1470           
Flag Coverage Δ
notify-server 89.13% <ø> (ø)
protocol-designer 45.85% <ø> (ø)
shared-data 86.02% <ø> (ø)
step-generation 88.46% <ø> (ø)

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

@sfoster1 sfoster1 merged commit 617a911 into edge Dec 13, 2022
y3rsh added a commit that referenced this pull request Dec 14, 2022
* origin/edge:
  fix(api): ot3controller should utilize `check_ready_for_movement` (#11752)
  chore(docs): updating Google Tag Manager on docs subdomain #11855
  refactor(app): add gripper attach, detach, and calibrate user-flow logic and skeletons (#11826)
  chore(build): fix push targets in api and s-d (#11879)
  fix(app): moveToLocation should be moveToMaintenancePosition (#11878)
  refactor(api): implement new Deck dependencies (#11853)
  refactor(api): implement new Deck collaborations (#11844)
  feat(app): create modals and add steps for 96 channel attach and detach (#11815)
  chore(build): Derive application versions from git (#11863)
  refactor(api): add support for Well.has_tip to the engine core (#11870)
  refactor(app, shared-data): calibration/moveToLocation to calibration/moveToMaintenancePosition (#11849)
  test(engine): add missing module_view test (#11872)
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.

3 participants