Skip to content

Conversation

@lucasborin
Copy link
Member

@lucasborin lucasborin commented Aug 12, 2021

2021-11-01 v.1.16.0

lucasborin and others added 30 commits August 12, 2021 14:08
* new check

* abapLint

* supporting programs

* closing gaps

* adding documentation

* Adding Changelog

* exempt ddic typed table

* supporting ddic table type

* supporting header line

* removing gap section

* supporting class attributes

* Use functional writing style for method calls

* Adding example
* new check

* new check

* documentation

* exempting SLIN

* Adding example
* Unit-Test Assert Validator 

resolve #449

* changelog
* BAdI example class vs Percentage Comment

resolve #443

* changelog
* big refactoring

* isolating unit tests

* changelog

* forcing commit

* forcing commit
* Functional operand check

* Fix unit test and nesting syntax

* Lint

* Update exemption unit test

* Fix unit test

* Add functional operand fix #460 to changelog

* removing static test, and adding more test cases

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.com>
* fix #461

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* saving progress

* saving progress

* Update changelog.txt

* global class vs local class

* local vs global test class

* Global Test Class fix

* enabling unit test

* refactoring y_check_function uts

* abaplint feedback

* abaplint feedback

* removing unecessary fields and fixing checksum

* fixing coverages (statement)

* default noaunit for test

* fixing bug in chain declaration

* fixing unit test

* Less than 3 newlines and at least a single newline are required in between metho

* ref #461

* solving conflict

* Update changelog.txt

* Update src/foundation/y_code_pal_ref_scan_double.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* Update src/foundation/y_code_pal_ref_scan_double.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* fixing collect dump

* fixing pseudo comment dump and performance issue

* reusing refscan (performance)

* Update src/checks/y_check_pseudo_comment_usage.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* Update src/checks/y_check_pseudo_comment_usage.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* Update src/checks/y_check_pseudo_comment_usage.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* Update src/checks/y_check_pseudo_comment_usage.clas.abap

Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>

* pseudo comment refactoring (performance)

* collect (performance)

* Update changelog.txt

* Reorder definitions to top of routine

* pseudo_comment_usage ignore_pseudo

* more performance improvements

* removing deprecated methods

* activating prefer_inline (#465)

* activating prefer_inline

* Prefer Inline Declarations, PERCENTAGE

* Prefer Inline Declarations, RISK_LVL

* Prefer Inline Declarations, HAS_REDIRECTION

* Prefer Inline Declarations, TITLE

* Prefer Inline Declarations, CLASS_SEARCH_STRING

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>

* performance improvement (check statement pos)

* Prefer Inline Declarations, PROFILE_CONFIGURATIONS

* Line break multiple parameters

* Reorder definitions to top of routine

* Update changelog.txt

* magic number

* adding pseudo comments

* Adding pseudo comment

* Fixing magic number

* Use | to assemble text!

* Use | to assemble text!

* Prefer RETURNING to EXPORTING!

* Prefer LINE_EXISTS or LINE_INDEX to READ TABLE or LOOP AT!

* Prefer IS NOT to NOT IS!

* replacing pseudo

* removing unecessary pseudos

* unecessary pseudo

* clean coding f4call

* new SAP note dependency

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
* resolves #474

* changelog

* forcing commit

* restore

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* resolves #474

* fix #475

* fixing case

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* Fixes #469

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* fixing magic number for leading zeros

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* case sy-tabix and Co numbers

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* Fixes #454

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* Fixes #332

* Missing space between string or character literal and parentheses

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* Fixes #484

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* saving progress

* saving progress

* saving progress

* alternative pseudo

* fixing 7.40 compatibility

* 7.40 compatibility

* revert checksum

* adjusting pseudo comment usage

* adding unit tests

* Missing space between string or character literal and parentheses

* new pseudo comment

* changelog

* alt_pcom in ut

* Update empty_catch.md

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* new test scope

* fixes #466

* cleaning self-reference

* cleaning chain declaration

* cleaning chain declaration

* cleaning chain declaration

* fixing syntax error

* changelog

* documentation

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* fixes #491

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* fixes #489

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* fixes #494

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* fixes #490

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* partially solves #433

* changelog

* renaming variable

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
lucasborin and others added 5 commits October 26, 2021 13:25
* fixes #433

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
* fixes #502

* changelog

Co-authored-by: Lucas Borin <5233413+lucasborin@users.noreply.github.co>
@lucasborin lucasborin marked this pull request as ready for review November 1, 2021 10:57
@lucasborin lucasborin requested a review from a team as a code owner November 1, 2021 10:57
@lucasborin lucasborin marked this pull request as draft November 1, 2021 10:58
@lucasborin lucasborin marked this pull request as ready for review November 1, 2021 11:43
@lucasborin
Copy link
Member Author

Tested in ABAP 7.40 = OK

@lucasborin lucasborin merged commit a86f9d1 into master Nov 1, 2021
@lucasborin lucasborin deleted the 1.16.0 branch November 1, 2021 13:06
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.

3 participants