Skip to content

Commit ba7f4f8

Browse files
LukasReschkeMorrisJobke
authored andcommitted
Improve phan config
* exclude routes * ignore some more templates of apps Signed-off-by: Lukas Reschke <lukas@statuscode.ch> Signed-off-by: Morris Jobke <hey@morrisjobke.de>
1 parent b3a0dec commit ba7f4f8

1 file changed

Lines changed: 61 additions & 7 deletions

File tree

build/.phan/config.php

Lines changed: 61 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,24 @@
1616
// Thus, both first-party and third-party code being used by
1717
// your application should be included in this list.
1818
'directory_list' => [
19-
'build/.phan/stubs',
2019
'3rdparty',
21-
'lib/composer',
22-
'themes',
23-
'lib/',
2420
'apps/',
21+
'build/.phan/stubs',
2522
'core/',
26-
'ocs/',
23+
'lib/',
24+
'lib/composer',
2725
'ocs-provider/',
26+
'ocs/',
2827
'settings/',
2928
'tests/lib/Util/User',
29+
'themes',
30+
],
31+
'file_list' => [
32+
'index.php',
33+
'public.php',
34+
'remote.php',
35+
'status.php',
36+
'version.php',
3037
],
3138

3239
// A directory list that defines files that will be excluded
@@ -40,56 +47,103 @@
4047
// party code, directories containing that code
4148
// should be added to the `directory_list` as
4249
// to `exclude_analysis_directory_list`.
43-
"exclude_analysis_directory_list" => [
50+
'exclude_analysis_directory_list' => [
4451
'3rdparty',
45-
'lib/composer',
4652
'apps/admin_audit/composer',
4753
'apps/admin_audit/tests',
4854
'apps/comments/composer',
4955
'apps/comments/tests',
5056
'apps/dav/composer',
57+
'apps/dav/templates',
5158
'apps/dav/tests',
5259
'apps/encryption/composer',
60+
'apps/encryption/templates/',
5361
'apps/encryption/tests',
5462
'apps/federatedfilesharing/composer',
63+
'apps/federatedfilesharing/templates/',
5564
'apps/federatedfilesharing/tests',
5665
'apps/federation/composer',
66+
'apps/federation/templates/',
5767
'apps/federation/tests',
5868
'apps/files/composer',
69+
'apps/files/templates/',
5970
'apps/files/tests',
6071
'apps/files_external/3rdparty',
6172
'apps/files_external/composer',
73+
'apps/files_external/templates/',
6274
'apps/files_external/tests',
6375
'apps/files_sharing/composer',
76+
'apps/files_sharing/templates/',
6477
'apps/files_sharing/tests',
6578
'apps/files_trashbin/composer',
79+
'apps/files_trashbin/templates/',
6680
'apps/files_trashbin/tests',
6781
'apps/files_versions/composer',
6882
'apps/files_versions/tests',
6983
'apps/lookup_server_connector/composer',
7084
'apps/lookup_server_connector/tests',
7185
'apps/oauth2/composer',
86+
'apps/oauth2/templates/',
7287
'apps/oauth2/tests',
7388
'apps/provisioning_api/composer',
7489
'apps/provisioning_api/tests',
7590
'apps/sharebymail/composer',
91+
'apps/sharebymail/templates/',
7692
'apps/sharebymail/tests',
7793
'apps/systemtags/composer',
94+
'apps/systemtags/templates',
7895
'apps/systemtags/tests',
7996
'apps/testing/composer',
8097
'apps/testing/tests',
8198
'apps/theming/composer',
99+
'apps/theming/templates/',
82100
'apps/theming/tests',
83101
'apps/twofactor_backupcodes/composer',
102+
'apps/twofactor_backupcodes/templates/',
84103
'apps/twofactor_backupcodes/tests',
85104
'apps/updatenotification/composer',
105+
'apps/updatenotification/templates/',
86106
'apps/updatenotification/tests',
87107
'apps/user_ldap/composer',
108+
'apps/user_ldap/templates/',
88109
'apps/user_ldap/tests',
89110
'apps/workflowengine/composer',
111+
'apps/workflowengine/templates/',
90112
'apps/workflowengine/tests',
113+
'build/.phan/',
114+
'core/templates',
115+
'lib/composer',
116+
'settings/templates',
91117
],
92118

119+
// A file list that defines files that will be excluded
120+
// from parsing and analysis and will not be read at all.
121+
//
122+
// This is useful for excluding hopelessly unanalyzable
123+
// files that can't be removed for whatever reason.
124+
'exclude_file_list' => [
125+
'apps/encryption/appinfo/routes.php',
126+
'apps/federation/appinfo/routes.php',
127+
'apps/files/appinfo/routes.php',
128+
'apps/files_external/appinfo/routes.php',
129+
'apps/files_trashbin/appinfo/routes.php',
130+
'apps/files_versions/appinfo/routes.php',
131+
'apps/testing/appinfo/routes.php',
132+
'apps/user_ldap/appinfo/routes.php',
133+
'settings/routes.php',
134+
'settings/ajax/updateapp.php',
135+
'settings/ajax/uninstallapp.php',
136+
'settings/ajax/togglesubadmins.php',
137+
'settings/ajax/setquota.php',
138+
'settings/ajax/enableapp.php',
139+
'settings/ajax/disableapp.php',
140+
'core/register_command.php',
141+
'ocs/routes.php',
142+
'ocs/v1.php',
143+
'core/routes.php',
144+
],
145+
146+
93147
// The number of processes to fork off during the analysis
94148
// phase.
95149
'processes' => 10,

0 commit comments

Comments
 (0)