github.com/TheAlgorithms/algorithms-keeper commits

Add mypy config to do strict checking

fa20268d3e89ef1fad25d894c4f6d37a56465ebb authored almost 4 years ago by Dhruv Manilawala <[email protected]>

Fix text not being bold due to lack of space

d98cee01ae81b636e4b3f8c963781c70d8d8745d authored almost 4 years ago by Dhruv Manilawala <[email protected]>

Fix multiple comments in the same body situation

ad0875c3cfc207e5450aae924030e55a5f60042c authored almost 4 years ago by Dhruv Manilawala <[email protected]>

Post all review data in one comment with review-all command

63d05addfa34c461d03f7bd25c87728359a6f4e1 authored almost 4 years ago by Dhruv Manilawala <[email protected]>

Refined types to be covariant for parameters

b0521d0c1d81930bb439d52c5116fcebce2a0254 authored almost 4 years ago by Dhruv Manilawala <[email protected]>

No need for the except block in remove labels

884f5a9b3c284c34ac3a8e737e2acf469d2687f8 authored almost 4 years ago by Dhruv Manilawala <[email protected]>

Update parser class with the class variables

378d6c6079a2236cc75e678cef8fc0a450fa1a05 authored almost 4 years ago by Dhruv Manilawala <[email protected]>

Ignore namedtuple assignment for name validation

99d2a1577390ed37c64c89e1ca3c5f52b78a956e authored almost 4 years ago by Dhruv Manilawala <[email protected]>

Fix naming convention rule to not complain for type alias

e8de66cf299040e15a213cf63ede5d778ace9158 authored almost 4 years ago by Dhruv Manilawala <[email protected]>

Small tweaks for type hints

716b185255116a818273ecf9c480ceb8d6b3c680 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Check attribute assignment for instances on self

5301f0a7a2129e80d8a22b5ec805332afb53ca56 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Check variable names for multiple assignment

c342138fbcbf1a29f3b81f9ce66bda98e1d2d9b5 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update the rule to support private class names

a44f6a920267a92a189de8477f1b260ee6c62cfd authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add useful comments and check parameter names for snake_case

c4225e3929b81b793cae3af4e6ec0467cf7b9fe0 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update naming convention rule to reduce the clutter

00e69c964bec19ffd95a626283f6f0cab8ad6d78 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update test rules function with error messages

76fb92d4e3ea9c4a368f5074c281d88a924cf889 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add rule for class/function/variable naming convention

cc46d4afe262066a36476c0c294a3417e904c158 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update the ability to collect the rules

We were collecting only the custom defined rules from the rules package.
We could not collect on...

4fb72f61e4ea1a24a65107dd5ed1ac75ded1edac authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix: update the number of rules in tests

d816ccb0db0ed0df50eb82de28d17f6b99b45c4e authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add reference link for f-string

19d0a2647e39764cce47c26f35bebdbd56461f86 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add rule to enforce the use of f-strings

15d589f6c68a32c08e5ffacecbdde9446544ba45 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update README

51f85c96ed853208700393cbe025e5c12ad596d1 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add ability to add/remove merge conflict label on PR

3e84146e0c028ea43413b2216bcf86da5d4f8249 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Lambda parameters cannot be annotated

88b9da2795d6f367d758ad29eeecc1dff0a6921d authored about 4 years ago by Dhruv Manilawala <[email protected]>

Bump libcst from 0.3.15 to 0.3.16 (#17)

Bumps [libcst](https://github.com/Instagram/LibCST) from 0.3.15 to 0.3.16.
- [Release notes](ht...

106d2dd0d1e11f3112482b7b669ebf0f03cdd8ed authored about 4 years ago by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Modify add error message for the record object

7bbfd8c211754b699640293cbd781bf6fcbc4642 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Bump pytest from 6.2.0 to 6.2.1 (#16)

Bumps [pytest](https://github.com/pytest-dev/pytest) from 6.2.0 to 6.2.1.
- [Release notes](htt...

45b7ec094b6d30d53d35dff26893b526126ce72c authored about 4 years ago by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Update README with the modified pull request event

5f75842d15f4b328c11eb36131b2277336a312ae authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fully annotated tests, use matchers for doctest rule

86b89f9befe8403b69c3f1011e687ed03aa297b5 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Separate out the logic for require and stage label

7abe9ba7d38d50cd0b2e72328d258c274b771b24 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add static typing for most of the test base

d8ad92a69a4df675fa56f7226a16222cb7708bd7 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update pull request tests with the new framework

ab90624cd259bb13b32c66b10e33cfa2464785c1 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Initial commit to change the testing framework

66de385b36900c6c3f07b9e0f8a21a5d94bd8a1b authored about 4 years ago by Dhruv Manilawala <[email protected]>

Remove redundant parameters for MockGitHubAPI object

bc23629510fd6867c77708db7113a3e7df50250e authored about 4 years ago by Dhruv Manilawala <[email protected]>

Remove testcase which used contains_testnode file

f14be25cc42a695cb6eab2ea1897809b22fbdbd4 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Remove testnode file as it is being tested

dd74bd097754a42824b7a8ef0c32a4e957777534 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix all tests with using the new fixit framework

d2d09aeabe1de3e6695a4e2e3c2edead2645c8f3 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add common test framework for testing rules

3c0b73800e75c3f2de386e627224743b0299e251 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Change logic of excluding DoctestRule when a test file is present

87ff452c49fcc48f98073ebcfd1979ad17db88be authored about 4 years ago by Dhruv Manilawala <[email protected]>

Major change: Use fixit framework to define rules

99fca97c61195aa90f7983891cf1ad63a0b4b5e2 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Use Monkeypatch from pytest public API

0d6208557b218782f90b3613fe4855489af2d71d authored about 4 years ago by Dhruv Manilawala <[email protected]>

Bump pytest from 6.1.2 to 6.2.0 (#15)

Bumps [pytest](https://github.com/pytest-dev/pytest) from 6.1.2 to 6.2.0.
- [Release notes](htt...

59feb98678eb747fba12c9823f8ddc0d92087d30 authored about 4 years ago by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Add error msg to comments and refined visitor

19d7b93d568fa89e81a45772844586f6efe90998 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Bump sentry-sdk from 0.19.4 to 0.19.5 (#14)

Bumps [sentry-sdk](https://github.com/getsentry/sentry-python) from 0.19.4 to 0.19.5.
- [Releas...

bf7bd89d12f6eaaaa571086b7586f3e6b50155ff authored about 4 years ago by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump cachetools from 4.1.1 to 4.2.0 (#13)

Bumps [cachetools](https://github.com/tkem/cachetools) from 4.1.1 to 4.2.0.
- [Release notes](h...

3ac6b9ecf323959dc762caa4494865ea475e0c50 authored about 4 years ago by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Ignore cls argument for the checks

83f8fed11cfb960c2b26321a83d7fee237c53771 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Redundant try finally in files to check generator

bce62e3cde681656648f9a088a14aa02fe23874d authored about 4 years ago by Dhruv Manilawala <[email protected]>

Constant name and place changes

67fc5239ed5a6d72ccb9d6c1f0a294bad397dd43 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix: parser label error and reorganized the parser

17e9a85c2efdb707f65f078e0d7b940b7d7e2a05 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add tests for group same line review comments

069a482eece194ec4cbc1fb1e1fbe189580748ac authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add ability to group comments from same lineno

f3a8045f44e58143230e90d1f9fe287c69bc8a0f authored about 4 years ago by Dhruv Manilawala <[email protected]>

Do not post review comments on review-all command

ed4bc0f0b2c6b47faeefac96f9bfab40aa717f77 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add command to trigger checks on all PR files

8a6824b8eed9f9cd7b19ca2ab267fca341163e4a authored about 4 years ago by Dhruv Manilawala <[email protected]>

Do not add type label if it already exists

deb47aa5d5691b474c1b82904ee65fbeb3beafd1 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update review body with the command info

5bad94ce979a6d272411c6903db0f92b1db1829e authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update README with bot commands

e5c1027a1c374dc7c7a970dcb282505788e046e8 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add tests for the review command

bc03df32ea827c458284e554d15444f7a9ac01f1 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Should not trigger when commented on issue

27403fee6be30192f50c188dac9e8380cdc2990a authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix: return the pull request object

454ebf10137c2e59ab1fd37af0f2a1891fb1e920 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add preview headers for Reaction API

4a1888c64408d1f8f03052bea1016ff5024e6a5d authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add commands router to the main router

680e4faba1cd8c14b11271aa994209e156487924 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Separate out the command function from the main

84d8b4c50d1111ec8cdf2e38c0dac8c85061f578 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add 'review' command for the bot

ae6f705677dadc6408bf2d542de411a506a96d46 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix: PR is not of type documentation when DIRECTORY.md is modified

0eee74f6b6a43b9b73ec18063d068a80e2b530d6 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix: include test for async function

36d1066713813dce83ab5a36ab28f0ab104bba62 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update visiting logic for the parser:

A class can contain doctest in the class-level docstring which
requires the visitor to skip doct...

d7b86754f0a463389f20c2347dba002d8c652e62 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Small tweaks to what is being logged

df30a079f8c58331b6d6d92a3bfa3e93b172d1f3 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update PR review body comment to hide link details

813c0369e13e5c951a35cf942dd98c7e045824b9 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix review labels not removed on add require label

a92e37a827cf60a5c14c5b0ed507eafbe4792432 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix label logic when all require labels are removed

d39625417e0cbd5a76b30b0ad06f52af2787471c authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix: Heroku assigns a PORT, fetch it from env

ddc9441285a8b804851e619dee927153523d3ca5 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Bump pre-commit from 2.9.2 to 2.9.3 (#12)

Bumps [pre-commit](https://github.com/pre-commit/pre-commit) from 2.9.2 to 2.9.3.
- [Release no...

260af52d36a4835113e5f4eea2540ed135cb482c authored about 4 years ago by dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Fix: CI shows failure for mypy strict analysis

4193a48d99d7a2e5ab584e766db0dc61641f64ab authored about 4 years ago by Dhruv Manilawala <[email protected]>

Small tweaks in logging and docs

72f5957444e0b3cfd77f55b3d65f71769959fe86 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Small tweaks in docs and code

9f0dc842428c325f3991f10a3e5c2a7d8a4a4195 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update color and style for mypy badge

2b8b45a5eeeb76072cd509265714edba5b8d7891 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add mypy badge to README.md

9b01e7d2ebaec811142b362c65e3ad5d80f14a76 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Run mypy outside pre-commit for strict analysis

3a8bb252348a1ecfd91cb6a31258e862618cd4bf authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update pre-commit hooks

4063c69a3b48fbd26d3e829a5d29380464462072 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Improve PR review body, access token logic change

08d7eb7f002fa592117ae0cd01d7045c215f10ea authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add tests for new parser and other changes:

- No cover for statements made for satisfying mypy
- Changed comment constant to INVALID_EXTENSI...

fc05f5625a5761c442f826d1fbc6f9d8a74427dc authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix mypy --strict errors

806959749c9d9c55627e6f9db0cfa0d692fecc09 authored about 4 years ago by Dhruv Manilawala <[email protected]>

fix mypy error for parser_rewrite and minor tweaks

5ab2b90c5f1db317e052d592d4947559ee20d1fb authored about 4 years ago by Dhruv Manilawala <[email protected]>

Use b64decode instead for decoding the file content

dec4f928e264e21c8d069a9ec26092eb7f8495e6 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Fix TypeError for TTLCache type annotation

70ed3410158642c62c457db663b493267e56fb1e authored about 4 years ago by Dhruv Manilawala <[email protected]>

Small tweaks and use absolute imports

7b2a4cbfeaddbfc9865210be9bc6c032b27d17a5 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Modified check_pr_files with the new parser

6156788d6a3ce5556f0045dd0ddc5eb749307356 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add function to create pr review

b231773176ab981b30d61f4640b62c9e95dc3d43 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update PR report constant to PR review body

8b98e4ca688b4a243f983e4d6647b4f140145b32 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Change model to Record, Parser and Visitor

e874c1f2bcd7ff4f4be68884960532a30470aec3 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Implemented Parser, Visitor and Reporter protocol

2d365669283c76a06fdd6d58cd7fd18f49e69f2f authored about 4 years ago by Dhruv Manilawala <[email protected]>

Clarified what name attr means for utils.File

603eb5f9d401e53c4691ccb843c2cf0901654273 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add type label constant and move extension const to parser

ae8a43724ceb81f1cb0c59d5281b4b351fa54662 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add ACCEPTED_EXTENSIONS constant for parser

b2c44540a1094c7de21b1641e12359d15b80fbd2 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add status attribute for File object

d4e075ae81317b1c5252441d19f9dbb30129d5bb authored about 4 years ago by Dhruv Manilawala <[email protected]>

Update requirements.txt with sentry_sdk package

02ca119e0e7d3aa9a7c52524650a06a84ee911ba authored about 4 years ago by Dhruv Manilawala <[email protected]>

Configure aiohttp integration

261783065e2534ffc055de14ff7c9cbfd4caf7d5 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Add aiohttp integration to sentry-sdk

01cb6027cb4f4b3cbb15ccf01f060e7c385c60d8 authored about 4 years ago by Dhruv Manilawala <[email protected]>

Trying out sentry_sdk for monitoring the app

0e855d2ddcb989824a37539214cf200ff55dd02f authored about 4 years ago by Dhruv Manilawala <[email protected]>

Remove all awaiting labels after PR is merged

02360bbcf1a64b4c414b56d461edc9baf146445f authored about 4 years ago by Dhruv Manilawala <[email protected]>

Ecosyste.ms: OpenCollective

github.com/TheAlgorithms/algorithms-keeper