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

[Core] Fix CI after poetry released 2.0.0 #1301

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Conversation

erikzaadi
Copy link
Member

Description

Explicitly limit our poetry to less than 2.0.0 since it breaks A LOT of our CI

Type of change

Please leave one option from the following and delete the rest:

  • Bug fix (non-breaking change which fixes an issue)

@github-actions github-actions bot added the size/M label Jan 7, 2025
@erikzaadi erikzaadi changed the title [CORE] Fix CI after poetry released 2.0.0 [Core] Fix CI after poetry released 2.0.0 Jan 7, 2025
Copy link
Member

@matan84 matan84 left a comment

Choose a reason for hiding this comment

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

LGTM

@erikzaadi erikzaadi force-pushed the PORT-12369-fix-poetry-ci branch from 94b0669 to e59582b Compare January 7, 2025 12:20
CHANGELOG.md Outdated
@@ -6,6 +6,15 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

<!-- towncrier release notes start -->

## 0.17.5 (2025-1-7)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## 0.17.5 (2025-1-7)
## 0.17.5 (2025-01-07)

@erikzaadi erikzaadi force-pushed the PORT-12369-fix-poetry-ci branch from d8e83ec to e4401a6 Compare January 7, 2025 14:38
@erikzaadi erikzaadi merged commit e4401a6 into main Jan 7, 2025
17 checks passed
@erikzaadi erikzaadi deleted the PORT-12369-fix-poetry-ci branch January 7, 2025 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants