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

Use FutureWarning instead of DeprecationWarning. #692

Merged
merged 14 commits into from
Mar 12, 2022
Merged

Use FutureWarning instead of DeprecationWarning. #692

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
daded12
Update __init__.py
Onkar627 Feb 26, 2022
234c34e
Update indexing.py
Onkar627 Feb 26, 2022
c318473
Merge branch 'glotzerlab:master' into patch-3
Onkar627 Mar 8, 2022
d51ec0c
Update collection_json.py
Onkar627 Mar 8, 2022
acdd77a
Update file_buffered_collection.py
Onkar627 Mar 8, 2022
5acba78
Update __init__.py
Onkar627 Mar 8, 2022
fcd0a6f
Merge branch 'glotzerlab:master' into patch-3
Onkar627 Mar 11, 2022
3472222
Update file_buffered_collection.py
Onkar627 Mar 11, 2022
9dad3e8
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Mar 11, 2022
57e0ccd
Update test_json_collection.py
Onkar627 Mar 11, 2022
55022f7
Update conftest.py
Onkar627 Mar 11, 2022
7cd1ea3
Removed deprecated_in_version function and use pytest.deprecated_call…
bdice Mar 12, 2022
ceacc11
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Mar 12, 2022
578f22f
Remove unused imports.
bdice Mar 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion signac/common/configobj/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1188,7 +1188,7 @@ def __init__(
warnings.warn(
"Passing in an options dictionary to ConfigObj() is "
"deprecated. Use **options instead.",
DeprecationWarning,
FutureWarning,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is vendored (copied from an upstream project) so we usually avoid changing it. We have control over how this object is created in the library and it's not a part of user-facing code, so I suggest reverting this change.

Suggested change
FutureWarning,
DeprecationWarning,

)

# TODO: check the values too.
Expand Down
2 changes: 1 addition & 1 deletion signac/contrib/indexing.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -620,7 +620,7 @@ def __init__(self, *args, **kwargs):
warnings.warn(
"The MasterCrawler class has been replaced by the MainCrawler class. "
"Both classes are deprecated and will be removed in a future release.",
DeprecationWarning,
FutureWarning,
)
super().__init__(*args, **kwargs)

Expand Down