-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fixture/v0.2.5 #10
Fixture/v0.2.5 #10
Conversation
…S. Including pep, PME and trsd in FileKind Enum. Modifying normalized version pattern to include .txa.
…age version from v0.2.4 to 0.2.5
…eader.read(...). Implementation of types str, StringIO and BytesIO in functions that uses FileReader.read(...) method
…the pep file for testing
… Including request warning in README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks like different behaviours of linters and automatic code fixes. Are you using ruff and black?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I didn't use Ruff and Black. I used Black, flake8 and mypy.
…ssed file local path when it is downloaded, to avoid files overwriting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wrote you some comments about your opinions. I made some changes which are available for your review and discussion.
Fixture v0.2.5 💜🏆
⚡ Implementation of trsd, pep and PEM file definitions.
⚡ Implementation of .txa file extension.
🛠️ Fix broken link in
/src/asic/publication.py -> ASIC_MONTHLY_VERSION_PUBLICATION_SERVICE["url"]
.🛠️ Fix mypy and flake8 issues.
🛠️ Fix file overwriting of preprocessed files downloaded from FPT server.