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

Fix Pylint errors in E packs #38053

Merged
merged 1 commit into from
Jan 9, 2025
Merged

Fix Pylint errors in E packs #38053

merged 1 commit into from
Jan 9, 2025

Conversation

RotemAmit
Copy link
Contributor

@RotemAmit RotemAmit commented Jan 8, 2025

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: link to the issue

Description

Fix Pylint errors in E packs.

Must have

  • Tests
  • Documentation

Copy link

github-actions bot commented Jan 8, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/EclecticIQ/Integrations/EclecticIQIntelligenceCenterv3
   EclecticIQIntelligenceCenterv3.py88523873%124, 127, 133, 169–172, 174, 178, 183, 187, 204, 231–238, 242–244, 246–248, 267–268, 270–271, 280–291, 293–296, 299–300, 303, 317, 322, 358–359, 731–732, 738–740, 748–750, 753, 757, 777, 790–792, 808, 813, 847–850, 855–857, 862, 864–865, 870, 872, 877, 1009, 1044–1045, 1050, 1061–1063, 1065–1066, 1087–1088, 1100–1101, 1160, 1182, 1190, 1196, 1199–1200, 1203–1206, 1209, 1218–1219, 1222, 1225, 1228, 1230–1231, 1243–1244, 1256–1258, 1260, 1299, 1304, 1306–1309, 1328, 1331, 1333, 1341, 1343–1346, 1349, 1351, 1386, 1392, 1394, 1396–1397, 1399–1400, 1402–1403, 1405–1406, 1408, 1410–1411, 1413–1414, 1416–1417, 1419, 1421, 1445, 1447, 1449, 1451, 1484, 1488, 1490–1493, 1496–1497, 1502–1503, 1555–1559, 1561–1563, 1565, 1587, 1596, 1662, 1672, 1677, 1687, 1707, 1717, 1732–1733, 1775, 1777, 1780, 1783–1784, 1787–1788, 1791–1792, 1795–1797, 1825, 1834, 1866, 1875, 1905, 1914, 1943, 1952, 2001, 2011, 2024, 2039, 2207, 2210, 2288, 2291, 2391–2392, 2416, 2418–2419, 2596, 2616–2617, 2619–2620, 2628–2630, 2632–2637, 2639–2640, 2644
Packs/Elasticsearch/Integrations/Elasticsearch_v2
   Elasticsearch_v2.py50518862%27–29, 44–45, 110, 149–152, 181, 185, 192, 253, 261, 314, 319, 327–329, 331, 333–334, 336–338, 340–341, 343–344, 348, 366–368, 370–371, 373–374, 376–377, 390–392, 394–395, 399, 401, 403–404, 413–415, 417–418, 422, 424, 426–427, 443–444, 446–447, 451, 453, 455, 457, 460, 475–476, 478–479, 483, 485, 487–488, 494, 504, 506–508, 510–511, 513–515, 517–518, 522, 525–526, 528–530, 533, 535–537, 539–540, 542, 547, 549–550, 552–553, 589–590, 592–593, 597, 599, 601, 603, 605, 608, 611, 614–619, 621, 623, 625–626, 629–630, 633–634, 638–639, 641–642, 646–647, 829–831, 833, 843–844, 846–847, 849–850, 852, 854–855, 857–858, 862, 864, 866, 868–871, 874–875, 877–878, 890, 958–965, 967–968, 972, 974–975, 977–979, 981, 999–1000, 1002, 1009, 1037, 1058–1059, 1061
Packs/EmailCommunication/Scripts/SendEmailReply
   SendEmailReply.py4608781%46, 122–123, 127, 149, 197, 215, 218, 240–243, 274, 309, 314, 373–374, 376–377, 381–392, 394, 466, 468, 471, 520–524, 542–543, 546, 560, 562, 564–569, 571, 573, 602–603, 611–612, 685, 769–770, 837–838, 880, 884, 893, 897–901, 905–909, 943, 954, 983, 1007, 1030, 1033, 1046–1047, 1089, 1110–1112
Packs/ExabeamDataLake/Integrations/ExabeamDataLake
   ExabeamDataLake.py801087%35–36, 48, 54, 152, 200, 203, 207–209
Packs/ExabeamSecurityOperationsPlatform/Integrations/ExabeamSecOpsPlatform
   ExabeamSecOpsPlatform.py3387478%108, 113–114, 120–122, 124, 152–153, 157, 163–166, 172–174, 180–182, 188–190, 196–198, 204–206, 212–214, 220–222, 241, 262, 395–397, 400, 421, 475, 497–498, 558, 569–570, 582–583, 599, 624, 635, 639–640, 642–643, 696, 754–755, 766, 801–806, 811–812, 814–816, 819, 858
TOTAL226859773% 

Tests Skipped Failures Errors Time
258 0 💤 0 ❌ 0 🔥 16.436s ⏱️

@RotemAmit
Copy link
Contributor Author

In ci/gitlab/gitlab.xdr.pan.local, the ExabeamDataLake-test fails - CRTX-149225

@RotemAmit RotemAmit added the ForceMerge Forcing the merge of the PR despite the build status label Jan 9, 2025
@RotemAmit
Copy link
Contributor Author

@dantavori Please force merge the PR. The reason for the it is written here #38053 (comment).

@dantavori dantavori merged commit 32aeed7 into master Jan 9, 2025
26 of 28 checks passed
@dantavori dantavori deleted the pylint_e branch January 9, 2025 08:45
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.

3 participants