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

Develop #460

Merged
merged 34 commits into from
Sep 20, 2022
Merged

Develop #460

merged 34 commits into from
Sep 20, 2022

Conversation

terazus
Copy link
Collaborator

@terazus terazus commented Sep 20, 2022

No description provided.

@terazus
Starting isatab refactoring
257328c
@terazus
fixed validator store add_info() parameters
64b3ef8
@terazus
Reduced code some more
178fe48
@terazus
minor style fixes
87107f3
@terazus
moved checks to sub module
825e3bb
@terazus
renamed checks to rules
60ee107
@terazus
refactored validator
752aed1
@terazus
added tests for new validate core
3a2592d
@terazus
Added performance CLI tool
d0114ed
@terazus
added docstring + ability to pass custom rules
fac69cb
@terazus
fixed minor error in isatab validator tests, confusing JS and python …
bdf4d05 
…with in and includes
@terazus
added function to build input rules dict for tab validators
11e7fbc
@terazus
extended performances CLI to isa json
0e7b403
@terazus
Added performances output profiles to gitignore
f0bedc9
@terazus
slightly readjusted performances architecture
db8784f
@terazus
slightly readjusted performances architecture
91363fb
@terazus
moved default performances variables to defaults.py
7f9f7fc
@terazus
moved all conversion tests to tests/convert
27b53e5
@terazus
moved deprecated rules one level lower
c96feda
@terazus
added load table checks (rule 4014) back in checks
ca9521d
@terazus
fixed error report for rule check_labels in load_investigation
b8226c7
@terazus
fixed previous commit
ea095df
@terazus
fixed rule 4014 to add message error to report
7f4fd8a
@terazus
added new test for reporting errors on non allowed headers
acc824a
@terazus
fixed missing importé
413fd29
@terazus
testing what ups with test data
49a5ec7
@terazus
changed mtlb input file
38a67bb
@terazus
Merge pull request #456 from ISA-tools/master
70de445 
Merged from master
@terazus
adding more meaningfull msg to rule 4014
6ff186f
@terazus
Adding missing coma in rule 4014
8f1935a
@terazus terazus marked this pull request as ready for review September 20, 2022 09:44
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 79.314% when pulling 00f4991 on develop into 6a5a530 on master.

@terazus terazus merged commit ca182d1 into master Sep 20, 2022
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.
2 participants
@terazus @coveralls