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

My name is debby Branch #932

Merged
merged 186 commits into from
Jul 16, 2022
Merged

My name is debby Branch #932

merged 186 commits into from
Jul 16, 2022

Conversation

atherdon
Copy link
Collaborator

@atherdon atherdon commented Jun 16, 2022

we don't need to merge it yet.

We need to rollback husky, so we can prevent stupid commits...

I can't say it's working 100% and bug-free, but I think I did the most work that I planned for it.

we still have issues with

  • header
  • preview
    other small elements must be tested like
  • italic
  • del
  • q
  • code
  • hr
  • empty
  • blockquote

Low Code Isn’t Dead Yet_ Or Is It_


Introduction to this Pull Request

video1374128521.mp4

Versions that I can reference to:


Video 2 with tests

video1235362811.mp4

@lgtm-com
Copy link

lgtm-com bot commented Jun 16, 2022

This pull request fixes 1 alert when merging 9643d44 into b644b27 - view on LGTM.com

fixed alerts:

  • 1 for Implicit operand conversion

@lgtm-com
Copy link

lgtm-com bot commented Jun 16, 2022

This pull request fixes 1 alert when merging 0a8b6bc into c61ac27 - view on LGTM.com

fixed alerts:

  • 1 for Implicit operand conversion

@lgtm-com
Copy link

lgtm-com bot commented Jun 16, 2022

This pull request fixes 1 alert when merging 7c9b0b6 into c61ac27 - view on LGTM.com

fixed alerts:

  • 1 for Implicit operand conversion

@lgtm-com
Copy link

lgtm-com bot commented Jun 16, 2022

This pull request introduces 1 alert when merging 83a57ef into c61ac27 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jun 21, 2022

This pull request introduces 1 alert when merging aebd1d2 into 10af54d - view on LGTM.com

new alerts:

  • 1 for Unreachable statement

@lgtm-com
Copy link

lgtm-com bot commented Jun 21, 2022

This pull request introduces 7 alerts when merging 64334c1 into 10af54d - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 1 for Unreachable statement

@lgtm-com
Copy link

lgtm-com bot commented Jun 21, 2022

This pull request introduces 10 alerts and fixes 2 when merging 72fd237 into 10af54d - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class
  • 2 for Implicit operand conversion
  • 2 for Unreachable statement

fixed alerts:

  • 2 for Implicit operand conversion

@lgtm-com
Copy link

lgtm-com bot commented Jun 21, 2022

This pull request introduces 13 alerts and fixes 2 when merging 504541c into 10af54d - view on LGTM.com

new alerts:

  • 8 for Unused variable, import, function or class
  • 2 for Implicit operand conversion
  • 2 for Unreachable statement
  • 1 for Conflicting function declarations

fixed alerts:

  • 2 for Implicit operand conversion

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 60 alerts and fixes 69 when merging 2e0b4ff into 7812eff - view on LGTM.com

new alerts:

  • 57 for Unused variable, import, function or class
  • 2 for Syntax error
  • 1 for Invocation of non-function

fixed alerts:

  • 42 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 67 alerts and fixes 76 when merging 7e029a7 into 7812eff - view on LGTM.com

new alerts:

  • 63 for Unused variable, import, function or class
  • 2 for Syntax error
  • 1 for Useless conditional
  • 1 for Invocation of non-function

fixed alerts:

  • 48 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 66 alerts and fixes 76 when merging 1af217c into 7812eff - view on LGTM.com

new alerts:

  • 58 for Unused variable, import, function or class
  • 6 for Syntax error
  • 1 for Useless conditional
  • 1 for Invocation of non-function

fixed alerts:

  • 48 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 52 alerts and fixes 76 when merging 105785c into 7812eff - view on LGTM.com

new alerts:

  • 50 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Invocation of non-function

fixed alerts:

  • 48 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 52 alerts and fixes 76 when merging 6f7ef2f into 7812eff - view on LGTM.com

new alerts:

  • 50 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Invocation of non-function

fixed alerts:

  • 48 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@atherdon atherdon marked this pull request as ready for review July 13, 2022 18:12
@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 51 alerts and fixes 76 when merging 2b2cbee into 7812eff - view on LGTM.com

new alerts:

  • 49 for Unused variable, import, function or class
  • 1 for Useless conditional
  • 1 for Invocation of non-function

fixed alerts:

  • 48 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 37 alerts and fixes 78 when merging 08bb17f into 7812eff - view on LGTM.com

new alerts:

  • 36 for Unused variable, import, function or class
  • 1 for Invocation of non-function

fixed alerts:

  • 50 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 34 alerts and fixes 78 when merging fb475b1 into 7812eff - view on LGTM.com

new alerts:

  • 33 for Unused variable, import, function or class
  • 1 for Invocation of non-function

fixed alerts:

  • 50 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@atherdon atherdon requested a review from vadim9999 July 13, 2022 18:37
@lgtm-com
Copy link

lgtm-com bot commented Jul 16, 2022

This pull request introduces 34 alerts and fixes 78 when merging 6cf93c6 into 7812eff - view on LGTM.com

new alerts:

  • 33 for Unused variable, import, function or class
  • 1 for Invocation of non-function

fixed alerts:

  • 50 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@lgtm-com
Copy link

lgtm-com bot commented Jul 16, 2022

This pull request introduces 34 alerts and fixes 78 when merging 57290da into 7812eff - view on LGTM.com

new alerts:

  • 33 for Unused variable, import, function or class
  • 1 for Invocation of non-function

fixed alerts:

  • 50 for Unused variable, import, function or class
  • 18 for Invocation of non-function
  • 7 for Implicit operand conversion
  • 1 for Useless conditional
  • 1 for Comparison between inconvertible types
  • 1 for Polynomial regular expression used on uncontrolled data

@vadim9999 vadim9999 merged commit d7d99a6 into main Jul 16, 2022
@vadim9999 vadim9999 deleted the my-name-is-debby branch July 17, 2022 07:28
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.

2 participants