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

Very generic naming of objects might cause clashes #360

Open
larshp opened this issue Apr 7, 2021 · 2 comments
Open

Very generic naming of objects might cause clashes #360

larshp opened this issue Apr 7, 2021 · 2 comments
Assignees
Labels
enhancement An enhancement or scope enhancement

Comments

@larshp
Copy link
Contributor

larshp commented Apr 7, 2021

code-pal has some objects with names that are very generic, like:
ycx_entry_not_found, y_alv_events, y_demo_failures

suggest introducing a naming scheme that reduces the possibility of name clashes in customer systems

@lucasborin
Copy link
Member

lucasborin commented Apr 7, 2021

Yeah, it would be good to have a default prefix on it all.

How about YPAL_ for the database, and Y_PAL_ for the ABAP?
(Y_PAL_IF_*)

Probably, it will require the renaming of the Checks:
Y_PAL_CHECK_<name>

And it will impact the current existing variants and profiles. Hm...

@lucasborin lucasborin added the enhancement An enhancement or scope enhancement label Apr 7, 2021
@larshp
Copy link
Contributor Author

larshp commented Apr 8, 2021

yea, and note that abaplint can help enforcing the naming pattern

@lucasborin lucasborin self-assigned this Dec 8, 2021
@lucasborin lucasborin linked a pull request Dec 8, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or scope enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants