Add file ignore for line too long errors

[ecalifornica]

Ignores linter error E501, continuation of #674 as I neglected to lint the tests/ path.

Unblocks #673

[reece]

Better: We can remove all of the future lines (in their entirety) from all files.
How about closing this PR without merging, then doing that?

[ecalifornica]

You read my mind, I was grepping for those __future__ imports earlier today. I've opened #688 to track that work.

This change is not targeting the __future__ module imports on the line below, but instead instructs ruff to ignore the line-too-long error for all lines in the file. Looks like the error is raised by the visual table example comments.

tests/test_hgvs_variantmapper_gcp.py:40:121: E501 Line too long (162 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:51:121: E501 Line too long (162 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:62:121: E501 Line too long (162 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:63:121: E501 Line too long (123 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:64:121: E501 Line too long (123 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:65:121: E501 Line too long (123 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:66:121: E501 Line too long (123 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:67:121: E501 Line too long (123 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:74:121: E501 Line too long (162 > 120 characters)
tests/test_hgvs_variantmapper_gcp.py:85:121: E501 Line too long (165 > 120 characters)

See also:
https://beta.ruff.rs/docs/configuration/#error-suppression

[reece]

This change is not targeting the __future__ module imports on the line below, but instead instructs ruff to ignore the line-too-long error for all lines in the file. Looks like the error is raised by the visual table example comments.

I misunderstood. I though the noqa pragma was for the future line. I now understand that it's for the whole file. Thanks.

[reece]

How about:

• set ruff line length = 100
• set black line length = 100
• reformat with make reformat
  ?

[ecalifornica]

How about:
* set ruff line length = 100
* set black line length = 100
* reformat with make reformat
?

Straightforward enough. Seems that black does not format comments or multi-line strings. Lowering the line-length configuration from 120 to 100 leaves 88 instances of the E501 Line too long error when ruff runs. These errors could be fixed by manually adding the noqa pragma directive to those lines.

[github-actions]

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

[github-actions]

This PR was closed because it has been stalled for 7 days with no activity.

[github-actions]

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

[github-actions]

This PR was closed because it has been stalled for 7 days with no activity.

[reece]

This issue was closed by stalebot. It has been reopened to give more time for community review. See biocommons coding guidelines for stale issue and pull request policies. This resurrection is expected to be a one-time event.

[github-actions]

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

[github-actions]

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

[github-actions]

This PR was closed because it has been stalled for 7 days with no activity.

[reece]

@ecalifornica : Thanks for contributing this. We're going to incorporate ideas from this PR into the template repo, and then propagate to project repos.