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

update/fix validations #92

Merged
merged 8 commits into from
Jul 26, 2024
Merged

update/fix validations #92

merged 8 commits into from
Jul 26, 2024

Conversation

mustafasoylu
Copy link
Collaborator

Update poetry, package.json and julia models and writer for person validation. Now, we dont raise an error when the person doesnt have email information, we ignore that input and print a warning message. This also fixes #87

closes #87

@mustafasoylu mustafasoylu added bug Something isn't working enhancement New feature or request labels Jul 23, 2024
@mustafasoylu mustafasoylu added this to the v0.4.3 milestone Jul 23, 2024
@mustafasoylu mustafasoylu self-assigned this Jul 23, 2024
@mustafasoylu mustafasoylu merged commit f42047b into main Jul 26, 2024
16 checks passed
@mustafasoylu mustafasoylu deleted the fix/validations branch July 26, 2024 13:15
@mustafasoylu mustafasoylu restored the fix/validations branch July 26, 2024 13:20
@mustafasoylu mustafasoylu deleted the fix/validations branch July 26, 2024 13:20
mustafasoylu added a commit that referenced this pull request Feb 5, 2025
commit b294b8c
Author: Mustafa SOYLU <[email protected]>
Date:   Wed Jan 15 10:55:45 2025 +0100

    bump version to 0.5.0 (#103)

commit 9c87008
Author: Mustafa SOYLU <[email protected]>
Date:   Wed Jan 15 10:41:47 2025 +0100

    update documentation (#94)

    * add notes

    * explain why set some fields as compulsory

    * Update README.md

    Co-authored-by: Volker Hofmann <[email protected]>

    ---------

    Co-authored-by: Volker Hofmann <[email protected]>

commit 42ae9c2
Author: Mustafa SOYLU <[email protected]>
Date:   Wed Jan 15 10:41:23 2025 +0100

    Optional email (#102)

    * make email optional in the person model

    * remove none's from the existing people list

    * make email optional in setuptools model

    * remove check of no email value since it is done in core now

    * update tests for optional email

    * show email is optional in manual

    * update doc dev docs for ruff usage

commit d28180d
Author: Mustafa Soylu <[email protected]>
Date:   Mon Jul 29 09:47:48 2024 +0200

    fix version in readme

commit 26dc24a
Merge: f42047b f53acd6
Author: Mustafa SOYLU <[email protected]>
Date:   Mon Jul 29 09:34:36 2024 +0200

    Merge pull request #93 from Materials-Data-Science-and-Informatics/release/v0.4.3

    v0.4.3

commit f53acd6
Author: Mustafa Soylu <[email protected]>
Date:   Mon Jul 29 09:25:27 2024 +0200

    update changelog and metadata

commit f42047b
Merge: a1548c9 78f8dc6
Author: Mustafa SOYLU <[email protected]>
Date:   Fri Jul 26 15:15:20 2024 +0200

    Merge pull request #92 from Materials-Data-Science-and-Informatics/fix/validations

    update/fix validations

commit 78f8dc6
Author: Mustafa Soylu <[email protected]>
Date:   Fri Jul 26 14:31:18 2024 +0200

    fix test since person conversion fails returns none on error

commit 4c81e63
Author: Mustafa Soylu <[email protected]>
Date:   Fri Jul 26 14:30:58 2024 +0200

    check if person if None on the base class

commit 993b714
Author: Mustafa Soylu <[email protected]>
Date:   Fri Jul 26 14:30:37 2024 +0200

    return None if person conversion fails

commit 188eda1
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:01:51 2024 +0200

    update changelog

commit 023c33b
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:01:33 2024 +0200

    add missing error type

commit acd3dab
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:01:18 2024 +0200

    disable error raise on users without email in julia

commit 32ad110
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:01:03 2024 +0200

    disable error raise on users without email in package.json

commit a400d97
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:00:19 2024 +0200

    disable error raise on users without email in poetry

commit a1548c9
Merge: 6257f3a 324b4c3
Author: Mustafa SOYLU <[email protected]>
Date:   Tue Jul 23 11:55:24 2024 +0200

    Merge pull request #91 from Materials-Data-Science-and-Informatics/feature/update_hooks

    update pre-commit hooks and dependencies

commit 324b4c3
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 11:41:52 2024 +0200

    update dependencies

commit 4bc902d
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 11:25:29 2024 +0200

    update pre-commit hooks

commit 6257f3a
Merge: 77c9345 0972820
Author: Mustafa SOYLU <[email protected]>
Date:   Tue Apr 30 11:39:15 2024 +0200

    Merge pull request #86 from Materials-Data-Science-and-Informatics/fix/log_format

    Fix/log format

commit 0972820
Author: Mustafa Soylu <[email protected]>
Date:   Tue Apr 30 11:35:01 2024 +0200

    bump version to 0.4.2

commit f02d8cf
Author: Mustafa Soylu <[email protected]>
Date:   Tue Apr 30 11:29:25 2024 +0200

    fix rich formating of error message and traceback

commit 77c9345
Merge: 16dfb47 8be7dd2
Author: Mustafa SOYLU <[email protected]>
Date:   Mon Apr 8 17:54:07 2024 +0200

    Merge pull request #82 from Materials-Data-Science-and-Informatics/fix/required_fields

    Fix/required fields

commit 8be7dd2
Author: Mustafa Soylu <[email protected]>
Date:   Mon Apr 8 14:13:58 2024 +0200

    bump version to 0.4.1

commit f48f5d3
Author: Mustafa Soylu <[email protected]>
Date:   Mon Apr 8 11:38:54 2024 +0200

    fix package license

commit 7bb381c
Author: Mustafa Soylu <[email protected]>
Date:   Mon Apr 8 11:33:37 2024 +0200

    fix mkdocs optional fields

commit c9d2179
Author: Mustafa Soylu <[email protected]>
Date:   Mon Apr 8 11:33:21 2024 +0200

    fix package json optional fields
mustafasoylu added a commit that referenced this pull request Feb 11, 2025
commit b294b8c
Author: Mustafa SOYLU <[email protected]>
Date:   Wed Jan 15 10:55:45 2025 +0100

    bump version to 0.5.0 (#103)

commit 9c87008
Author: Mustafa SOYLU <[email protected]>
Date:   Wed Jan 15 10:41:47 2025 +0100

    update documentation (#94)

    * add notes

    * explain why set some fields as compulsory

    * Update README.md

    Co-authored-by: Volker Hofmann <[email protected]>

    ---------

    Co-authored-by: Volker Hofmann <[email protected]>

commit 42ae9c2
Author: Mustafa SOYLU <[email protected]>
Date:   Wed Jan 15 10:41:23 2025 +0100

    Optional email (#102)

    * make email optional in the person model

    * remove none's from the existing people list

    * make email optional in setuptools model

    * remove check of no email value since it is done in core now

    * update tests for optional email

    * show email is optional in manual

    * update doc dev docs for ruff usage

commit d28180d
Author: Mustafa Soylu <[email protected]>
Date:   Mon Jul 29 09:47:48 2024 +0200

    fix version in readme

commit 26dc24a
Merge: f42047b f53acd6
Author: Mustafa SOYLU <[email protected]>
Date:   Mon Jul 29 09:34:36 2024 +0200

    Merge pull request #93 from Materials-Data-Science-and-Informatics/release/v0.4.3

    v0.4.3

commit f53acd6
Author: Mustafa Soylu <[email protected]>
Date:   Mon Jul 29 09:25:27 2024 +0200

    update changelog and metadata

commit f42047b
Merge: a1548c9 78f8dc6
Author: Mustafa SOYLU <[email protected]>
Date:   Fri Jul 26 15:15:20 2024 +0200

    Merge pull request #92 from Materials-Data-Science-and-Informatics/fix/validations

    update/fix validations

commit 78f8dc6
Author: Mustafa Soylu <[email protected]>
Date:   Fri Jul 26 14:31:18 2024 +0200

    fix test since person conversion fails returns none on error

commit 4c81e63
Author: Mustafa Soylu <[email protected]>
Date:   Fri Jul 26 14:30:58 2024 +0200

    check if person if None on the base class

commit 993b714
Author: Mustafa Soylu <[email protected]>
Date:   Fri Jul 26 14:30:37 2024 +0200

    return None if person conversion fails

commit 188eda1
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:01:51 2024 +0200

    update changelog

commit 023c33b
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:01:33 2024 +0200

    add missing error type

commit acd3dab
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:01:18 2024 +0200

    disable error raise on users without email in julia

commit 32ad110
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:01:03 2024 +0200

    disable error raise on users without email in package.json

commit a400d97
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 16:00:19 2024 +0200

    disable error raise on users without email in poetry

commit a1548c9
Merge: 6257f3a 324b4c3
Author: Mustafa SOYLU <[email protected]>
Date:   Tue Jul 23 11:55:24 2024 +0200

    Merge pull request #91 from Materials-Data-Science-and-Informatics/feature/update_hooks

    update pre-commit hooks and dependencies

commit 324b4c3
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 11:41:52 2024 +0200

    update dependencies

commit 4bc902d
Author: Mustafa Soylu <[email protected]>
Date:   Tue Jul 23 11:25:29 2024 +0200

    update pre-commit hooks

commit 6257f3a
Merge: 77c9345 0972820
Author: Mustafa SOYLU <[email protected]>
Date:   Tue Apr 30 11:39:15 2024 +0200

    Merge pull request #86 from Materials-Data-Science-and-Informatics/fix/log_format

    Fix/log format

commit 0972820
Author: Mustafa Soylu <[email protected]>
Date:   Tue Apr 30 11:35:01 2024 +0200

    bump version to 0.4.2

commit f02d8cf
Author: Mustafa Soylu <[email protected]>
Date:   Tue Apr 30 11:29:25 2024 +0200

    fix rich formating of error message and traceback

commit 77c9345
Merge: 16dfb47 8be7dd2
Author: Mustafa SOYLU <[email protected]>
Date:   Mon Apr 8 17:54:07 2024 +0200

    Merge pull request #82 from Materials-Data-Science-and-Informatics/fix/required_fields

    Fix/required fields

commit 8be7dd2
Author: Mustafa Soylu <[email protected]>
Date:   Mon Apr 8 14:13:58 2024 +0200

    bump version to 0.4.1

commit f48f5d3
Author: Mustafa Soylu <[email protected]>
Date:   Mon Apr 8 11:38:54 2024 +0200

    fix package license

commit 7bb381c
Author: Mustafa Soylu <[email protected]>
Date:   Mon Apr 8 11:33:37 2024 +0200

    fix mkdocs optional fields

commit c9d2179
Author: Mustafa Soylu <[email protected]>
Date:   Mon Apr 8 11:33:21 2024 +0200

    fix package json optional fields
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Optional fields treated as required
1 participant