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

Tails zero warnings #30290

Merged
merged 56 commits into from
Oct 22, 2023
Merged

Tails zero warnings #30290

merged 56 commits into from
Oct 22, 2023

Conversation

thefrieddan1
Copy link
Contributor

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: https://jira-hq.paloaltonetworks.local/browse/CIAC-2332

Description

Removing warnings from our repo

@DeanArbel DeanArbel merged commit 52f2be4 into master Oct 22, 2023
11 of 13 checks passed
@DeanArbel DeanArbel deleted the tails_zero_warnings branch October 22, 2023 12:02
@DeanArbel DeanArbel added the ForceMerge Forcing the merge of the PR despite the build status label Oct 22, 2023
@DeanArbel
Copy link
Contributor

Test playbook failures are unrelated to changes, approved for force merge.

sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* fix PytestUnhandledThreadExceptionWarning warning
in UrlScan

* PytestUnhandledThreadExceptionWarning version.

* handle PytzUsageWarning.

* UserWarning: DataFrame

* handle PytestMockWarning

* SettingWithCopyWarning

* Handle PytestReturnNotNoneWarning

* Handle GuessedAtParserWarning

* Handle PytestUnraisableExceptionWarning

* Handle InsecureRequestWarning.

* Handle PytestUnknownMarkWarning

* Handle PytestRemovedIn8Warning

* Handling warnings.

* trigger nightly

* reduce warnings.

* reduce warnings.

* More warnings.

* Add ignore warnings to pyproject.toml file

* testing if commit nightly can run now

* Fix wrong usage of ignore warning in pyproject.

* yet again fix warnings.

* Remove pytets changes from pyproject.toml

* Trying to ignore using a pytest.ini
in root of a pack.

* Remove pytest.ini for specific pack.

* ignore PytestUnraisableExceptionWarning

* Ignore marshmallow and ignore unknown mark
warnings.

* Use exact path to PytestUnknownMarkWarning

* Yet another method to try ignore
PytestUnknownMarkWarning

* resolve conflicts.

* Versions and readme and versions update.

* Remove "Note: Zero warnings effort." from RN.

* Update to latest docker images.

* revert wrong changes.

* Fix failing unit tests.

* making changes to catch
linter validations

* try to load Axonius src file in lint stage.

* specify type for args.

* Bump pack from version Phishing to 3.5.28.

* import marshmallow and ignore marshmallow warning.

* Break long lines.

* Different ignore

* Update docker images.

* Bump pack from version CommonScripts to 1.12.37.

* update forgotten docker images.

* Fix RN validation errors.

* yet another docker image.

---------

Co-authored-by: Content Bot <bot@demisto.com>
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.

2 participants