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

demisto-sdk-release 1.33.4 #37948

Merged
merged 2 commits into from
Jan 7, 2025
Merged

demisto-sdk-release 1.33.4 #37948

merged 2 commits into from
Jan 7, 2025

Conversation

content-bot
Copy link
Collaborator

@content-bot content-bot commented Jan 6, 2025

Breaking

  • Started deprecation process for old validate. This flow will be deprecated and removed in the near future. Please use the new validate flow. #4750

Feature

  • Added support for validating CaseLayout content items in the RN114 validation. #4755
  • Updated the ST110 validation to validate the "sectionOrder" and individual sections. Added new validation ST111 which validates the existence of the same fields. #4739
    #4735

Fix

  • Fixed an issue where the build-Devcontainer Github action failed due to file_path being None in setup-environment command. #4747
  • Fixed an issue where an incorrect marketplace reference was not properly removed from the code. #4745

@CLAassistant
Copy link

CLAassistant commented Jan 7, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ barryyosi-panw
❌ Content Bot


Content Bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@barryyosi-panw
Copy link
Contributor

Hi @DeanArbel, force merge is required here as of the known pylint error in pre-commit, in addition to the poetry version issue in the devcontainer which is handled by on-call in #37955 (poetry has released a breaking version 2.0.0 so currently #37955 locks poetry version on an earlier one - @mmhw awaits the pylint issue to be fixed before merging the devcontainer fix)

@barryyosi-panw barryyosi-panw added the ForceMerge Forcing the merge of the PR despite the build status label Jan 7, 2025
@DeanArbel DeanArbel merged commit de6d535 into master Jan 7, 2025
18 of 21 checks passed
@DeanArbel DeanArbel deleted the 1.33.4 branch January 7, 2025 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-approved ForceMerge Forcing the merge of the PR despite the build status
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants