Add current context support with push/pop mechanism#7
Conversation
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
JohnAmadis
left a comment
There was a problem hiding this comment.
@copilot małe poprawki
| * @param ctx Context to set as default | ||
| * @deprecated Use dmenv_set_root_context instead | ||
| */ | ||
| DMOD_BUILTIN_API(dmenv, 1.0, void, _set_as_default, (dmenv_ctx_t ctx)); |
There was a problem hiding this comment.
Te funkcje deprecated możesz usunąć od razu, ja poprawie odwołania w drugim repo
There was a problem hiding this comment.
Removed dmenv_set_as_default and dmenv_get_default functions and updated tests in commit 31eb055.
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
|
@copilot readme nie zaktualizowales |
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Updated README with new context stack API documentation in commit d8be37a. Added:
|
|
@copilot jakiś problem z kompilacja jest na CI CMake Error: Not a file: /memory_analysis.cmake |
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
|
@copilot jeszcze to: |
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Fixed in commit a3b53ba. The coverage job was failing because the custom linker script (for embedded targets) prevents
|
g_default_contexttog_root_contextand update API names:dmenv_set_root_contextAPIdmenv_get_root_contextAPIdmenv_set_as_defaultanddmenv_get_defaultfunctionsdmenv_push_contextAPIdmenv_pop_contextAPIdmenv_get_current_contextAPI to get the current context (top of stack or root)Dmod_SetEnvandDmod_GetEnvto use current contextOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.