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

Add lint #15

Merged
merged 45 commits into from
Aug 28, 2023
Merged

Add lint #15

merged 45 commits into from
Aug 28, 2023

Conversation

hoxbro
Copy link
Member

@hoxbro hoxbro commented Aug 9, 2023 •
edited
Loading

Uses pre-commit to lint code. Code changes are fixed linting.

hoxbro added 30 commits June 29, 2023 18:41
@hoxbro
Rewrite TestBasicRange1DAnnotator to pytest
78730df
@hoxbro
Rewrite TestBasicRange2DAnnotator to pytest
37e8ab2
@hoxbro
Convert test_table_region_df to pytest
309b6e4
@hoxbro
Run ruff --select=pt
0fc2624
@hoxbro
Update TestBasicPoint1DAnnotator
8f5bbd9
@hoxbro
Update TestBasicPoint2DAnnotator
4c40e2b
@hoxbro
Split up into basic and advanced
c707e32
@hoxbro
Rewrite test_connectors
ee08b18
@hoxbro
Remove commented out code
266ce67
@hoxbro
Update test_annotator_advanced
e2347b9
@hoxbro
Update test to give all output
674b4ba
@hoxbro
update database fields
7bd831c
@hoxbro
Refactor pytest fixtures
1fa0813
@hoxbro
DRY fixtures
23fef76
@hoxbro
Clean up test_connectors
cfe5e01
@hoxbro
Remove database in tests
42c25e1
@hoxbro
Duplicate behavior
d2d8a66
@hoxbro
Support all database for test_add_three_rows_delete_one
bd4bf38 
Also set key_list=None
@hoxbro
Merge branch 'use_pytest' into fix_duplicate
1d49c19
@hoxbro
Remove duplicate code in test and remove list in init
cf55cb4
@hoxbro
Move loading of annotation_table into its class
6bd72c0
@hoxbro
Add init
c4d48bb
@hoxbro
Remove mutable input
37ae038
@hoxbro
Add return_commits
7c2c6ca
@hoxbro
Fix multiple_region_annotator test
236baa2
@hoxbro
Fix test_multiplot_add_annotation and remove annotation_tables from c…
f714169 
…onnector
@hoxbro
Don't initialize connector
4a38c19
@hoxbro
Combine test_reconnect
c02b412
@hoxbro
Remove all drop_duplicates
79a8efe
@hoxbro
Clean up
fddae93
@hoxbro hoxbro marked this pull request as draft August 18, 2023 10:50
Base automatically changed from fix_duplicate to main August 28, 2023 15:47
@hoxbro hoxbro marked this pull request as ready for review August 28, 2023 15:54
@hoxbro hoxbro merged commit 32de68f into main Aug 28, 2023
10 checks passed
@hoxbro hoxbro deleted the add_lint branch August 28, 2023 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hoxbro