starred/cypht

Fork 0

mirror of https://github.com/cypht-org/cypht.git synced 2026-04-25 13:05:53 +03:00

[GH-ISSUE #962] Hm, TM - what are these in reference to? #520

New issue

Closed

opened 2026-02-25 21:35:14 +03:00 by kerem · 9 comments

kerem commented

2026-02-25 21:35:14 +03:00

Owner

Originally created by @TheDigitalOrchard on GitHub (Apr 18, 2024).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/962

💬 Question

Throughout the Cypht framework, there are classes prefixed with Hm_, and the Environment variables are stored with a prefix of TM_.

What are these in reference to? Legacy name? If so, can we deprecate these in favour of a Cypht_ \ CYPHT_ prefix?

Originally created by @TheDigitalOrchard on GitHub (Apr 18, 2024). Original GitHub issue: https://github.com/cypht-org/cypht/issues/962 ## 💬 Question Throughout the Cypht framework, there are classes prefixed with `Hm_`, and the Environment variables are stored with a prefix of `TM_`. What are these in reference to? Legacy name? If so, can we deprecate these in favour of a `Cypht_ \ CYPHT_` prefix?

kerem closed this issue

2026-02-25 21:35:14 +03:00

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@marclaporte commented on GitHub (Apr 18, 2024):

Context: https://github.com/cypht-org/cypht/issues/602

Yes, we should clean up!

If low risk, we sneak in 2.0, else 2.1 or 3.0

@marclaporte commented on GitHub (Apr 18, 2024): Context: https://github.com/cypht-org/cypht/issues/602 Yes, we should clean up! If low risk, we sneak in 2.0, else 2.1 or 3.0

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@marclaporte commented on GitHub (Apr 18, 2024):

@Shadow243 As you worked on https://github.com/cypht-org/cypht/pull/823/ please opine.

@marclaporte commented on GitHub (Apr 18, 2024): @Shadow243 As you worked on https://github.com/cypht-org/cypht/pull/823/ please opine.

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@Shadow243 commented on GitHub (Apr 18, 2024):

@Shadow243 As you worked on #823 please opine.

This is an option to take into consideration, however there is too much refactoring to do because this will affect all cypht modules, unit tests, lib and certain javascript files.

I would therefore suggest doing this in a version other than 2.0 which must already be released asap.

@Shadow243 commented on GitHub (Apr 18, 2024): > @Shadow243 As you worked on #823 please opine. This is an option to take into consideration, however there is too much refactoring to do because this will affect all cypht modules, unit tests, lib and certain javascript files. I would therefore suggest doing this in a version other than 2.0 which must already be released asap.

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@marclaporte commented on GitHub (Apr 18, 2024):

and the Environment variables are stored with a prefix of TM_.

@Shadow243 what does TM stand for?

@marclaporte commented on GitHub (Apr 18, 2024): > and the Environment variables are stored with a prefix of `TM_`. @Shadow243 what does TM stand for?

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@Shadow243 commented on GitHub (Apr 18, 2024):

TM

The idea of putting dotenv came from tiki manager, this TM surely came from there too (TIKI MANAGER) and so I didn't change it by putting something else, it could be CYPHT_

@Shadow243 commented on GitHub (Apr 18, 2024): > TM The idea of putting dotenv came from tiki manager, this TM surely came from there too (TIKI MANAGER) and so I didn't change it by putting something else, it could be CYPHT_

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@marclaporte commented on GitHub (Apr 18, 2024):

This is an option to take into consideration, however there is too much refactoring to do because this will affect all cypht modules, unit tests, lib and certain javascript files.

Is this for Hm_, TM_ or both?

Cypht 2.0 has not been released yet so almost no one is using TM_ yet. So it could make sense to do before we release 2.0: https://github.com/cypht-org/cypht/issues/966

@marclaporte commented on GitHub (Apr 18, 2024): > This is an option to take into consideration, however there is too much refactoring to do because this will affect all cypht modules, unit tests, lib and certain javascript files. Is this for `Hm_`, `TM_` or both? Cypht 2.0 has not been released yet so almost no one is using TM_ yet. So it could make sense to do before we release 2.0: https://github.com/cypht-org/cypht/issues/966

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@Shadow243 commented on GitHub (Apr 18, 2024):

This is an option to take into consideration, however there is too much refactoring to do because this will affect all cypht modules, unit tests, lib and certain javascript files.

Is this for Hm_, TM_ or both?

Cypht 2.0 has not been released yet so almost no one is using TM_ yet. So it could make sense to do before we release 2.0: https://github.com/cypht-org/cypht/issues/966

Copy that @marclaporte

@Shadow243 commented on GitHub (Apr 18, 2024): > > This is an option to take into consideration, however there is too much refactoring to do because this will affect all cypht modules, unit tests, lib and certain javascript files. > > > > Is this for `Hm_`, `TM_` or both? > > > > Cypht 2.0 has not been released yet so almost no one is using TM_ yet. So it could make sense to do before we release 2.0: https://github.com/cypht-org/cypht/issues/966 > > > > > > Copy that @marclaporte

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@marclaporte commented on GitHub (Apr 18, 2024):

https://github.com/cypht-org/cypht/pull/967

@marclaporte commented on GitHub (Apr 18, 2024): https://github.com/cypht-org/cypht/pull/967

kerem commented

2026-02-25 21:35:15 +03:00

Author

Owner

@marclaporte commented on GitHub (May 6, 2024):

So this task is partly done, and the rest is covered by https://github.com/cypht-org/cypht/issues/602

@marclaporte commented on GitHub (May 6, 2024): So this task is partly done, and the rest is covered by https://github.com/cypht-org/cypht/issues/602

kerem referenced this issue

2026-02-25 21:36:28 +03:00

[PR #520] [MERGED] Remove an extra character ')' #869