Skip to content
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

maint: misc. refactorings, fixes, updates #111

Merged
merged 1 commit into from
Dec 3, 2022

Conversation

yozachar
Copy link
Collaborator

  • puts tests in the correct location of github workflow
  • fixes .env values not loading via load_dotenv - required early loading
  • corrects many static type linting errors
  • combines all inputs into a single class with validation
  • formats markdown & python files, as well as output
  • slightly improved log messages, caught potential attribute error
  • updates dependencies

@yozachar yozachar self-assigned this Nov 30, 2022
@yozachar yozachar force-pushed the workshop branch 2 times, most recently from ce906d7 to 9a35587 Compare November 30, 2022 08:50
@yozachar yozachar added documentation Improvements or additions to documentation maintenance Maintenance Work dependencies Pull requests that update a dependency file labels Nov 30, 2022
@yozachar yozachar force-pushed the workshop branch 3 times, most recently from af05353 to a71072a Compare November 30, 2022 09:36
- puts tests in the correct location of github workflow
- fixes `.env` values not loading via `load_dotenv` - required early loading
- corrects many static type linting errors
- combines all inputs into a single class with validation
- formats markdown & python files, as well as output
- slightly improved log messages, caught potential attribute error
- updates dependencies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation maintenance Maintenance Work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant