diff --git a/.copier-answers.yml b/.copier-answers.yml index fe78ceed44..d8c39bf830 100644 --- a/.copier-answers.yml +++ b/.copier-answers.yml @@ -1,5 +1,5 @@ # Do NOT update manually; changes here will be overwritten by Copier -_commit: v1.19 +_commit: v1.21.1 _src_path: gh:oca/oca-addons-repo-template additional_ruff_rules: [] ci: GitHub diff --git a/.github/workflows/stale.yml b/.github/workflows/stale.yml index 1693a1253b..fa17fcd4e8 100644 --- a/.github/workflows/stale.yml +++ b/.github/workflows/stale.yml @@ -9,7 +9,7 @@ jobs: runs-on: ubuntu-latest steps: - name: Stale PRs and issues policy - uses: actions/stale@v4 + uses: actions/stale@v9 with: repo-token: ${{ secrets.GITHUB_TOKEN }} # General settings. @@ -48,7 +48,7 @@ jobs: # * Issues that are pending more information # * Except Issues marked as "no stale" - name: Needs more information stale issues policy - uses: actions/stale@v4 + uses: actions/stale@v9 with: repo-token: ${{ secrets.GITHUB_TOKEN }} ascending: true diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 36c22a87b3..03c1acbbcf 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -63,7 +63,9 @@ jobs: run: oca_init_test_database - name: Run tests run: oca_run_tests - - uses: codecov/codecov-action@v1 + - uses: codecov/codecov-action@v4 + with: + token: ${{ secrets.CODECOV_TOKEN }} - name: Update .pot files run: oca_export_and_push_pot https://x-access-token:${{ secrets.GIT_PUSH_TOKEN }}@github.com/${{ github.repository }} if: ${{ matrix.makepot == 'true' && github.event_name == 'push' && github.repository_owner == 'OCA' }} diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index d96af4c699..0aba465c95 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -16,6 +16,8 @@ exclude: | readme/.*\.(rst|md)$| # Ignore build and dist directories in addons /build/|/dist/| + # Ignore test files in addons + /tests/samples/.*| # You don't usually want a bot to modify your legal texts (LICENSE.*|COPYING.*) default_language_version: @@ -41,7 +43,7 @@ repos: hooks: - id: whool-init - repo: https://github.com/oca/maintainer-tools - rev: f71041f22b8cd68cf7c77b73a14ca8d8cd190a60 + rev: 9a170331575a265c092ee6b24b845ec508e8ef75 hooks: # update the NOT INSTALLABLE ADDONS section above - id: oca-update-pre-commit-excluded-addons @@ -110,7 +112,7 @@ repos: args: [--fix, --exit-non-zero-on-fix] - id: ruff-format - repo: https://github.com/OCA/pylint-odoo - rev: v8.0.19 + rev: v9.0.4 hooks: - id: pylint_odoo name: pylint with optional checks diff --git a/.ruff.toml b/.ruff.toml index 634db8e79b..0240c75f6a 100644 --- a/.ruff.toml +++ b/.ruff.toml @@ -6,6 +6,7 @@ fix = true extend-select = [ "B", "C90", + "E501", # line too long (default 88) "I", # isort "UP", # pyupgrade ] diff --git a/fs_attachment/models/ir_attachment.py b/fs_attachment/models/ir_attachment.py index 7bafad546f..d026fc88f8 100644 --- a/fs_attachment/models/ir_attachment.py +++ b/fs_attachment/models/ir_attachment.py @@ -295,7 +295,8 @@ def write(self, vals): # nor the mimetype, so the mimetype will be set to ``application/octet- # stream``. # We want to avoid this, so we take the mimetype of the first attachment - # and we set it on all the attachments if they all have the same mimetype. + # and we set it on all the attachments if they all have the same + # mimetype. # If they don't have the same mimetype, we raise an error. # OPW-3277070 mimetypes = self.mapped("mimetype") @@ -560,7 +561,9 @@ def _fs_mark_for_gc(self, fname): def _get_fs_parts( self, ) -> tuple[fsspec.AbstractFileSystem, str, str] | tuple[None, None, None]: - """Return the filesystem, the storage code and the path for the current attachment""" + """Return the filesystem, the storage code and the path for the + current attachment + """ if not self.store_fname: return None, None, None return self._fs_parse_store_fname(self.store_fname) @@ -1069,7 +1072,8 @@ def _get_attachment_data(self) -> dict: file_path = file_path.lstrip("/") ret["checksum"] = self._filesystem.checksum(file_path) ret["file_size"] = self._filesystem.size(file_path) - # TODO index_content is too expensive to compute here or should be configurable + # TODO index_content is too expensive to compute here or should be + # configurable # data = self._file.read() # ret["index_content"] = self.attachment._index_content(data, # self.attachment.mimetype, ret["checksum"]) diff --git a/fs_image/fields.py b/fs_image/fields.py index fe1b813edd..1513f6aad4 100644 --- a/fs_image/fields.py +++ b/fs_image/fields.py @@ -219,7 +219,8 @@ def _update_alt_text(self, records, value: dict): if not record[self.name]: raise UserError( _( - "Cannot set alt_text on empty image (record %(record)s.%(field_name)s)", + "Cannot set alt_text on empty image " + "(record %(record)s.%(field_name)s)", record=record, field_name=self.name, )