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.

Sign up for GitHub

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

Jump to bottom

Checklist for 'reformat' PR #235

Closed
5 of 6 tasks
jkbhagatio opened this issue Aug 27, 2023 · 1 comment
Closed
5 of 6 tasks

Checklist for 'reformat' PR #235

jkbhagatio opened this issue Aug 27, 2023 · 1 comment
Assignees
@jkbhagatio

Comments

@jkbhagatio
Copy link
Member

jkbhagatio commented Aug 27, 2023 •
edited
Loading

After merging 'config' (#228) into 'main', we will create another branch called 'reformat', which will contain all the reformatted changes according to the specs in 'config'.

We should make sure the following works / is done before merging 'reformat' into 'main':

  • .pre-commit-config.yaml
  • build_env_run_tests.yml
  • gpt_pr_review.yml
  • squash_merge_to_prod.yml
  • Update branch protection rules
  • Create docstrings for all modules and functions
@jkbhagatio jkbhagatio self-assigned this Aug 27, 2023
@jkbhagatio
Copy link
Member Author

Create docstrings for all modules and functions

This last one will be done in "cleanup" branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jkbhagatio jkbhagatio

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jkbhagatio