Commit Graph

88 Commits

Author SHA1 Message Date
Lars Hvam
d0a7f97dda
additional forbidden void types (#4532) 2021-02-18 15:53:59 +01:00
Lars Hvam
ed89d2dd7c
update abaplint.json, dont define the same constant twice (#4531) 2021-02-18 14:55:11 +01:00
Lars Hvam
5efc7a63ab
update deps in package.json (#4514)
* update deps in package.json

* enable rule max_one_method_parameter_per_line
2021-02-15 14:16:16 +01:00
Lars Hvam
8e07d46f4b
jump_se11, remove unused parameters (#4498) 2021-02-09 15:50:27 +01:00
Lars Hvam
a704c355aa
unit testing test (#4391)
* unit testing test

* upd

* upd

* upd

* upd

* update deps

* upd

* upd

* update deps

* upd

* update abaplint to latest

* enable rule use_class_based_exceptions
2021-01-12 15:57:17 +01:00
Lars Hvam
0003d8b85e
update ajson with writer (#4255)
* update ajson with writer

* update to latest

* fix exceptions

* convert exception
2020-12-18 09:55:57 +01:00
Lars Hvam
260af61407
ecatt exceptions, test (#4312) 2020-12-12 08:51:49 +01:00
Lars Hvam
0a24986682
update linter configuration to latest (#4241)
* update linter configuration to latest

* rerun

* rerun
2020-11-24 17:07:14 +01:00
Lars Hvam
a53776ab5a
delete unused method in ZCL_ABAPGIT_OBJECT_CHDO (#4130)
* delete unused method

delete unused method GET_GENERIC in class zcl_abapgit_object_chdo

* enable rule unused_methods
2020-11-03 14:55:01 +01:00
Mike Pokraka
c1b2cf10a0
Update unit tests to use CI (#4106)
* Update unit tests to use CI

* Make test repo configurable

* Remove local tests and use CI

* Lint fixes

* Replace local tests with CI

* Add Type Group constants to abaplint

* Update constants and replace assert_true

* Revert constants

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-03 08:49:22 +01:00
Lars Hvam
bda8280ba0
abaplint: enable parser_702_chaining rule (#4121) 2020-11-02 14:51:47 +01:00
Marc Bernard
bdc44f7476
Remove self-references (#4115)
* Remove self-references

Remove unnecessary self-references (me->)

https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#omit-the-self-reference-me-when-calling-an-instance-method

* 702: Fix syntax error

No chaining with EXPORTING & CHANGING

* remove addiontal me, update abaplint config

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-11-02 09:25:27 +01:00
Lars Hvam
ff962f82e8
remove INTERFACE LOAD statement (#4035) 2020-10-17 11:07:28 +02:00
Lars Hvam
45bc9ed91d
update linter configuration (#4008) 2020-10-11 10:40:18 +02:00
Marc Bernard
9672f5b030
Adjust name prefix for types (#3991)
* Part 1

* Part 2

* Part 3

* Lint

* Part 5

* Part 6

* Part 7

* Part 8

* Part 9

* Enforce type naming prefix

All types need to start with TY_...

* Fix local types
2020-10-07 08:54:15 +02:00
Lars Hvam
1c1b04fbe9
update lint config, minor refactors (#3948) 2020-09-27 09:24:31 +02:00
Lars Hvam
03724484f1
use boolc (#3920) 2020-09-23 12:28:13 +02:00
Lars Hvam
015800b45b
Less DDIC references (#3905)
* remove uses of CHAR30

* Update abaplint.json

* add basic unit test for ZCL_ABAPGIT_MESSAGE_HELPER

* replace uses of char50

* fix name

* Update abaplint.json

* use sy-uname instead of uname

in ZCL_ABAPGIT_USER_MASTER_RECORD
2020-09-21 07:39:22 +02:00
Alexander Tsybulsky
d20cdd07c2
syst_title is missing in 731 (#3896)
* syst_title is mising in 731

* adjust type

* Update abaplint.json

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-09-17 17:40:38 +02:00
Lars Hvam
b4169ff4b6
update lint config, plus remove identical condition (#3892) 2020-09-16 15:51:12 +02:00
Lars Hvam
42a532b9eb
reduce usage of ddic types (#3859)
* add SAP_BOOL as forbidden void type

* reduce

* Update abaplint.json

* sychar10

* sylangu

* Update abaplint.json
2020-09-10 15:30:25 +02:00
Lars Hvam
cbb35ade88
linter: add numc2 as forbidden void type (#3837) 2020-09-04 09:36:31 +02:00
Lars Hvam
428575fb58
1.99.1 (#3832)
* v1.99.1

* 1.99.1

* linter, enable rule line_break_style
2020-09-03 09:30:44 +02:00
Lars Hvam
9c41a2b7d8
remove text pragmas and pseudo comments (#3827)
* remove text pragmas and pseudo comments

* fix whitespace

* fix carriage return

* test

* test

* Update zcl_abapgit_repo.clas.abap
2020-09-03 09:13:47 +02:00
Lars Hvam
75b83825ed
update linter config (#3822) 2020-09-01 16:27:16 +02:00
Alexander Tsybulsky
a4e6c87985
Switching to PRs (#3695)
* string map: public entries (read only)

* port http agent

* draft wip

* drafting

* fix SSL

* port ajson

* integrate ajson and http_agent

* agent global headers

* github enumerator

* update ajson

* draft wip

* enumerator factory

* http_agent via factory

* proxy

* repo->switch_origin draft

* refactor enum factory

* Revert "refactor enum factory"

This reverts commit bcfef40b38991adfe5f73677919db6bd0c652f7c.

* page wiring draft

* scripts draft

* "let it work!"

* repo setings page draft

* linter

* linter

* linter

* linter

* linter

* better settings page

* finetune page

* cleanups

* change hotkey

* fix ajson_error

* Update src/zcl_abapgit_repo_online.clas.abap

Co-authored-by: Lars Hvam <larshp@hotmail.com>

* abaplint nesting 6

* remove description

* back to good old popups

* linter

* move commands to branches dropdown

* auth helpers

* move json to separate package

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-08-27 07:47:27 +02:00
Lars Hvam
e88f522e83
OTGR: syntax error, disable parents temporarily (#3787)
* OTGR: syntax error, disable parents temporarily

* Update abaplint.json
2020-08-24 11:36:58 +02:00
pokrakam
b20a020637
Allow naming of different test class types (#3771)
* Allow naming of different test class types

Propose to use the more commonly used naming conventions for test classes rather than naming everything ltcl_. These are used by SAP and also referenced in the style guides e.g. the section on [Helper Methods](https://github.com/SAP/styleguides/blob/master/clean-abap/CleanABAP.md#put-help-methods-in-help-classes) and used in the ABAP unit testing book.

ltc_: test class
lth_: test helper
ltd_: test double
lts_: test suite

Oh, and also corrected bug in exception class name that I spotted when editing

* Update with generic naming suggestions

Co-authored-by: Lars Hvam <larshp@hotmail.com>

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-08-19 08:41:55 +02:00
pokrakam
138172ffda
Exclude test class with XML from line length check (#3755)
* Exclude test class with XML from line length check

We could also exclude all zcl_abapgit_object*testclasses or even test classes in general.

* Correct syntax

* Update abaplint.json

* Update abaplint.json

Co-authored-by: Lars Hvam <larshp@hotmail.com>
2020-08-17 20:37:18 +02:00
Lars Hvam
d1f277365b
create_sotr, remove unused parameters (#3739)
* minor changes

* create_sotr, remove unused parameters
2020-08-16 08:21:49 +02:00
Lars Hvam
7cf1202723
update linter configuration (#3734) 2020-08-14 12:22:32 +02:00
Lars Hvam
867c74701f
remove some unused variables (#3719)
* remove some unused variables

* more unused
2020-08-09 08:44:54 +02:00
Lars Hvam
37ec998898
update lint config (#3618) 2020-07-13 10:22:33 +02:00
Lars Hvam
74ea9f046a
linting: errors for anything that contains "ABAPGIT" (#3523) 2020-06-18 17:03:23 +02:00
Lars Hvam
024167ce97
replace SYREPID (#3501)
* Update abaplint.json

* replace SYREPID
2020-06-16 07:34:12 +02:00
Lars Hvam
9af61c99c6
fully type sydatum and syuzeit (#3457) 2020-06-07 09:01:08 +02:00
Lars Hvam
d36b43defe
line break parameters (#3444)
* line break parameters

* Update zcl_abapgit_gui_page_repo_sett.clas.abap

* Update zcl_abapgit_gui_page_repo_sett.clas.abap

* fix
2020-06-04 15:10:35 +02:00
Lars Hvam
ea03cc5e4a
Fewer DDIC type references (#3434)
* Fewer DDIC type references

* Update abaplint.json
2020-05-29 06:30:24 +02:00
Lars Hvam
aec7468aad
Type Hunting (#3414)
* Type Hunting

* update

* update

* flag
2020-05-26 05:55:28 +02:00
Lars Hvam
f431d30140
setup linting of the compiled report (#3408)
* lint the compiled report

* change

* Update ci.yml
2020-05-24 10:36:14 +02:00
Lars Hvam
f17eddfd6b linting, update error namespace 2020-05-23 11:46:36 +02:00
larshp
323789836e add icons 2020-05-18 07:20:06 +02:00
larshp
0308182f81 update lint config 2020-05-16 10:39:15 +02:00
g-back
07c87edd03
PINF: Exporting to Returning Parameter (#3252)
* change exporting to returning parameter

* enable prefer_returning_to_exporting
2020-03-11 11:42:15 +01:00
Lars Hvam
136dfaf8d3 disable ambiguous_statement rule
its not possible to make MODIFY explicit on 702 syntax
2020-02-08 14:34:45 +01:00
Lars Hvam
7d0503218e no macros #3170 2020-02-03 09:20:58 +01:00
Lars Hvam
38837c3268
zlib, remove macros, #3170 (#3190)
* zlib, remove macros, #3170

* sneaky update to abaplint.json

* sneaky sneaky

* Update abaplint.json
2020-02-02 07:01:56 +01:00
Lars Hvam
3e0b0505b1
remove commented code (#3166)
* update abaplint.json

* remove commented code

* remove 2 unused variables
2020-01-24 14:31:04 +01:00
larshp
54f8f41c22 update abaplint.json 2020-01-11 09:41:52 +01:00
Lars Hvam
75812fb17b
less prefixes (#3144) 2020-01-11 09:24:36 +01:00