Skip to content

Fix a few issues with code style#71

Merged
ziotom78 merged 8 commits intomasterfrom
fix68
Oct 25, 2020
Merged

Fix a few issues with code style#71
ziotom78 merged 8 commits intomasterfrom
fix68

Conversation

@ziotom78
Copy link
Copy Markdown
Member

This PR addresses issue #68 :

  • Run black on all the source files
  • Fix every error and warning produced by flake8
  • Prevent tests from polluting the litebird_sim directory with temporary files.

No functionality has been added or changed to the framework.

A few test cases wrote files containing numbers in the base
`litebird_sim` folder. This commit adds data files containing
the expected numerical values and changes the test code to
actually ensure that the result of these calculations agree
with the reference numbers.
This commits forces tests to create their files in a temporary
directory instead of the base source directory.
@ziotom78 ziotom78 merged commit c829ca7 into master Oct 25, 2020
@ziotom78 ziotom78 deleted the fix68 branch October 25, 2020 11:41
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