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

add lint deprecation message. #4286

Merged
merged 7 commits into from
May 21, 2024
Merged

add lint deprecation message. #4286

merged 7 commits into from
May 21, 2024

Conversation

YuvHayun
Copy link
Contributor

Related Issues

related to: https://jira-dc.paloaltonetworks.com/browse/CIAC-10009

Description

Add a deprecation message to the demisto-sdk lint command.
Remove lint from all flows mentioned in sdk.

@YuvHayun YuvHayun requested a review from samuelFain as a code owner May 20, 2024 08:44
@github-actions GitHub Actions
Copy link

Changelog(s) in markdown:

  • Removed lint step from all sdk build steps. #4286
  • Started adaption for demisto-sdk lint command. The command will be fully removed soon. #4286

JudahSchwartz
JudahSchwartz reviewed May 20, 2024
View reviewed changes
.changelog/4286.yml Show resolved Hide resolved
.changelog/4286.yml Outdated Show resolved Hide resolved
demisto_sdk/commands/lint/lint_manager.py Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

Changelog(s) in markdown:

  • Removed lint step from all sdk build steps. #4286
  • Started adaption for demisto-sdk lint command. The command will be fully removed soon Use demisto-sdk pre-commit instead. #4286

@YuvHayun @JudahSchwartz
Update demisto_sdk/commands/lint/lint_manager.py
06ec12d 
Co-authored-by: Judah Schwartz <JudahSchwartz@users.noreply.github.com>
@YuvHayun YuvHayun merged commit 9db89e4 into master May 21, 2024
26 checks passed
@YuvHayun YuvHayun deleted the document_lint_deprecation branch May 21, 2024 09:31
@content-bot content-bot mentioned this pull request May 28, 2024
Merged
@israelpoli israelpoli mentioned this pull request May 28, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JudahSchwartz JudahSchwartz JudahSchwartz approved these changes

@samuelFain samuelFain Awaiting requested review from samuelFain samuelFain is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@YuvHayun @JudahSchwartz