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

Add (silent) support for Sorbet comments #966

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

matteolc
Copy link

@matteolc matteolc commented Aug 4, 2022

Add (silent) support for Sorbet comments, # typed: ...

@matteolc
Copy link
Author

matteolc commented Aug 4, 2022

Not sure why this CI test is failing (is green locally)

image

Add Sorbet comment matcher

Signed-off-by: Matteo La Cognata <matteo.lacognata@capdesk.com>
Signed-off-by: Matteo La Cognata <matteo.lacognata@capdesk.com>
Signed-off-by: Matteo La Cognata <matteo.lacognata@capdesk.com>
commit 22ab676
Author: Cuong Tran <ctran@users.noreply.github.com>
Date:   Thu Mar 30 15:17:37 2023 -0700

    chore: remove broken badges from README.md

commit 10a7a76
Author: Takumi KAGIYAMA <694547+kg8m@users.noreply.github.com>
Date:   Fri Mar 31 07:15:07 2023 +0900

    Support `--frozen` option for routing annotations (ctran#979)

    The `--frozen` option previously deal only model annotations.  This change will support route annotations as well.

    ---------

    Signed-off-by: kg8m <takumi.kagiyama@gmail.com>
    Co-authored-by: Cuong Tran <ctran@users.noreply.github.com>

commit a28fef3
Author: Jim Jowdy <jjowdy@gmail.com>
Date:   Wed Mar 29 15:09:11 2023 -0700

    Fix retrieve_indexes_from_table when indexes is empty and base table does not exist. (ctran#849)

    Some tables may have a table_name_prefix but no indexes. Previous versions of
    the code would strip the prefix and look for indexes on the resulting table
    which likely would not exist. This causes DB errors, at least in MySQL. So now
    check if the new table exists first before trying to show its indexes.

commit 13b532d
Author: Cuong Tran <ctran@users.noreply.github.com>
Date:   Wed Mar 29 01:51:15 2023 -0700

    Update codeql-analysis.yml

commit ea4cd00
Author: Lovro Bikić <lovro.bikic@infinum.hr>
Date:   Wed Mar 29 10:31:01 2023 +0200

    Add support for annotating check constraints (ctran#868)

    This adds annotation of check constraints with an option to disable/enable annotation. Most of the work done in this PR is based off of existing implementation for annotating indexes and foreign keys.

    Signed-off-by: Lovro Bikic <lovro.bikic@infinum.hr>

commit 76a1804
Author: Lovro Bikić <lovro.bikic@gmail.com>
Date:   Wed Mar 29 10:18:24 2023 +0200

    Fix flaky specs (ctran#980)

    Signed-off-by: Lovro Bikic <lovro.bikic@infinum.hr>
drwl added a commit to drwl/annotaterb that referenced this pull request May 23, 2023
Add (silent) support for Sorbet comments, `# typed: ...`.

From ctran/annotate_models#966. Credit goes to
@matteolc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant