Skip to content

Commit

Permalink
Add deprecation TODOs for future us
Browse files Browse the repository at this point in the history
Signed-off-by: Jean-Christophe Morin <[email protected]>
  • Loading branch information
JeanChristopheMorinPerso committed Nov 7, 2023
1 parent 2d1fa3e commit b621823
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/rezplugins/package_repository/filesystem.py
Original file line number Diff line number Diff line change
Expand Up @@ -208,6 +208,7 @@ def _load(self):

return data

# TODO: Deprecate
def _load_old_formats(self):
data = None

Expand Down Expand Up @@ -245,6 +246,7 @@ def _update_changelog(file_format, data):
if not maxlen:
return data

# TODO: Deprecate
if file_format == FileFormat.yaml:
changelog = data.get("changelog")
if changelog:
Expand Down Expand Up @@ -334,6 +336,7 @@ def iter_packages(self):
yield package

def _load(self):
# TODO: Deprecate: What is self.ext?
format_ = FileFormat[self.ext]
data = load_from_file(
self.filepath,
Expand Down Expand Up @@ -433,6 +436,7 @@ def _root(self):
class FileSystemPackageRepository(PackageRepository):
"""A filesystem-based package repository.
TODO: Deprecate YAML
Packages are stored on disk, in either 'package.yaml' or 'package.py' files.
These files are stored into an organised directory structure like so:
Expand Down Expand Up @@ -1182,6 +1186,7 @@ def _get_file(self, path, package_filename=None):
package_filenames = _settings.package_filenames

for name in package_filenames:
# TODO: Deprecate YAML
for format_ in (FileFormat.py, FileFormat.yaml):
filename = "%s.%s" % (name, format_.extension)
filepath = os.path.join(path, filename)
Expand Down

0 comments on commit b621823

Please sign in to comment.