Skip to content

Fix a csvimport error 'List index (0) out of bounds'#120

Merged
skupriienko merged 1 commit intomasterfrom
fix-csvimport
Jul 11, 2025
Merged

Fix a csvimport error 'List index (0) out of bounds'#120
skupriienko merged 1 commit intomasterfrom
fix-csvimport

Conversation

@skupriienko
Copy link
Collaborator

Actions:

  • Fix a csvimport error 'List index (0) out of bounds' (see https://app.mailjet.com/contacts):

    • rename json_data back to data. The bug was introduced in v1.4.0.
  • CI:

    • Update pre-commit hooks
    • Ignore refurb rule FURB184
  • Tests:

    • Add class TestCsvImport(unittest.TestCase) with a test suite for testing CSV import functionality to test.py. You need an additional environment variable ID_CONTACTSLIST taken from My contact lists https://app.mailjet.com/contacts
    • Add tests/doc_tests/files/data.csv
  • Package management:

    • Add the ruff rule PLC0207 to the ignore list

@skupriienko skupriienko self-assigned this Jul 11, 2025
@skupriienko skupriienko marked this pull request as ready for review July 11, 2025 10:20
@skupriienko skupriienko merged commit cec3ac9 into master Jul 11, 2025
16 checks passed
@skupriienko skupriienko deleted the fix-csvimport branch July 11, 2025 10:21
@skupriienko skupriienko changed the title fix: Fix a csvimport error 'List index (0) out of bounds' Fix a csvimport error 'List index (0) out of bounds' Jul 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant