add if __main__ to changelog filter #338
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
changelog_filter.py
script added in #329 is causing issues with test collection. This can be seen in downstream tests in asdfhttps://github.com/asdf-format/asdf/actions/runs/8065305926/job/22030878789?pr=1745
and by attempting to run
pytest
in a development install of dkist:This PR adds an
if __name__ == '__main__'
check to the script so it does not attempt to read stdin during test collection.