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

Merge from master #601

Merged
merged 521 commits into from
Dec 31, 2024
Merged

Merge from master #601

merged 521 commits into from
Dec 31, 2024

Conversation

ikostan
Copy link
Member

@ikostan ikostan commented Dec 31, 2024

No description provided.

suggestion (code-quality): Convert for loop into list comprehension (list-comprehension)
Trailing newlines

kyu_8/logical_calculator/
test_logical_calculator_error.py

62
            self.assertEqual(str(calc_err.value), err)
Unnecessary parens after 'with' keyword

kyu_8/logical_calculator/
test_logical_calculator_error.py

51
            "</p>")
52
        # pylint: enable=R0801
53
        with (allure.step("Pass an array with invalid operator.")):
54
            arr: list = []
55
            op: str = 'RO'  # invalid operator
Unused print_log imported from utils.log_func

kyu_8/logical_calculator/
test_logical_calculator_error.py

13
from utils.log_func import print_log
Merge pull request #575 from iKostanOrg/master
./kyu_7/complete_the_pattern_5_even_ladder/solution.py:8:1: E302 expected 2 blank lines, found 1
def pattern(n: int) -> str:
^
1     E302 expected 2 blank lines, found 1
kyu_8/logical_calculator/test_logical_calculator_error.py:35 in public method `test_logical_calc_value_error`:
        D205: 1 blank line required between summary line and description (found 0)

        Multi-line docstrings consist of a summary line just like a one-line
        docstring, followed by a blank line, followed by a more elaborate
        description. The summary line may be used by automatic indexing tools;
        it is important that it fits on one line and is separated from the
        rest of the docstring by a blank line.
Merge pull request #577 from iKostanOrg/master
Merge pull request #579 from iKostanOrg/master
 Check notice on line 1 in kyu_8/find_the_first_non_consecutive_number/__init__.py

@codacy-production
codacy-production
/ Codacy Static Code Analysis
kyu_8/find_the_first_non_consecutive_number/__init__.py#L1

Final newline missing
checking consistency... /home/docs/checkouts/readthedocs.org/user_builds/codewars/checkouts/582/docs/kyu_2/kyu_2.readme.rst: WARNING: document isn't included in any toctree
/home/docs/checkouts/readthedocs.org/user_builds/codewars/checkouts/582/docs/kyu_3/kyu_3.readme.rst: WARNING: document isn't included in any toctree
/home/docs/checkouts/readthedocs.org/user_builds/codewars/checkouts/582/docs/kyu_4/kyu_4.readme.rst: WARNING: document isn't included in any toctree
/home/docs/checkouts/readthedocs.org/user_builds/codewars/checkouts/582/docs/kyu_5/kyu_5.readme.rst: WARNING: document isn't included in any toctree
/home/docs/checkouts/readthedocs.org/user_builds/codewars/checkouts/582/docs/kyu_6/kyu_6.readme.rst: WARNING: document isn't included in any toctree
/home/docs/checkouts/readthedocs.org/user_builds/codewars/checkouts/582/docs/kyu_7/kyu_7.readme.rst: WARNING: document isn't included in any toctree
/home/docs/checkouts/readthedocs.org/user_builds/codewars/checkouts/582/docs/kyu_8/kyu_8.readme.rst: WARNING: document isn't included in any toctree
ikostan and others added 22 commits December 30, 2024 06:23
Merge pull request #597 from iKostanOrg/master
Bumps [actions/checkout](https://github.com/actions/checkout) from 2 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v2...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [DavidAnson/markdownlint-cli2-action](https://github.com/davidanson/markdownlint-cli2-action) from 18 to 19.
- [Release notes](https://github.com/davidanson/markdownlint-cli2-action/releases)
- [Commits](DavidAnson/markdownlint-cli2-action@v18...v19)

---
updated-dependencies:
- dependency-name: DavidAnson/markdownlint-cli2-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…-github/workflows/actions/checkout-4

Bump actions/checkout from 2 to 4 in /.github/workflows
…-github/workflows/DavidAnson/markdownlint-cli2-action-19

Bump DavidAnson/markdownlint-cli2-action from 18 to 19 in /.github/workflows
@ikostan ikostan self-assigned this Dec 31, 2024
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for 8f7dce01
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (8f7dce0) Report Missing Report Missing Report Missing
Head commit (a795878) 2602 2356 90.55%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#601) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

The pull request #601 has too many files changed.

We can only review pull requests with up to 300 changed files, and this pull request has 8421.

@ikostan ikostan merged commit 5106b3a into kyu6 Dec 31, 2024
26 of 29 checks passed
ikostan added a commit that referenced this pull request Dec 31, 2024
Merge pull request #601 from iKostanOrg/master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant