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.

Sign up for GitHub

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

Jump to bottom

change pyyaml constraint #12679

Merged
merged 1 commit into from
Jul 19, 2023
Merged

change pyyaml constraint #12679

merged 1 commit into from
Jul 19, 2023

Conversation

vcidst
Copy link
Contributor

@vcidst vcidst commented Jul 19, 2023

Proposed changes:

  • Change the PyYAML constraint to resolve the CI issues. This will unblock multiple PRs in rasa-oss and rasa-plus

Status (please check what you already did):

  • added some tests for the functionality
  • updated the documentation
  • updated the changelog (please check changelog for instructions)
  • reformat files using black (please check Readme for instructions)

@vcidst
Copy link
Contributor Author

vcidst commented Jul 19, 2023

@Urkem I think #12678 might take some time to be merged to master. So I've created a different PR directly to master. Once this is merged I'll close the other one

@vcidst vcidst enabled auto-merge (squash) July 19, 2023 13:42
@vcidst vcidst requested a review from Urkem July 19, 2023 13:43
@vcidst vcidst merged commit 95ff31d into main Jul 19, 2023
95 checks passed
@vcidst vcidst deleted the upgrade-pyyaml branch July 19, 2023 14:22
@vcidst vcidst mentioned this pull request Jul 19, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Urkem Urkem Urkem approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vcidst @Urkem