Skip to content

Repo/main correction#41

Merged
GuilhermeF03 merged 7 commits into
mainfrom
repo/main-correction
Mar 14, 2026
Merged

Repo/main correction#41
GuilhermeF03 merged 7 commits into
mainfrom
repo/main-correction

Conversation

@GuilhermeF03

Copy link
Copy Markdown
Member

Summary
This PR serves to correct discrepancies between main and branch 31, on features that were out of the branch scope

Changes

  • Moved 'create' block into 'enter tree' lifecycle method, instead of 'on ready'
  • Some fixes to the contexts
  • Simplified transitive dependency graph
  • Added contributor-oriented comments

Type of Change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Tests
  • Chore
  • Breaking change

Checklist

  • Code follows project style
  • Self-reviewed
  • Tests added/updated (if applicable)
  • Docs updated (if applicable)

Allows end-users to import less modules
* Refactored logging module for better organization
* Made banner logging bare console print
* Moved it into "nodeEnterTree" instead of "nodeReady"
* Also made context provide lambdas, so calling "provide" calls the up to date value
@rcosta358

Copy link
Copy Markdown
Collaborator

@copilot review

@GuilhermeF03

GuilhermeF03 commented Mar 14, 2026

Copy link
Copy Markdown
Member Author

Logging-related changes will be overwritten by branch 31's PR

@GuilhermeF03 GuilhermeF03 merged commit 2ec2673 into main Mar 14, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in canopy-tasks Mar 14, 2026
@GuilhermeF03 GuilhermeF03 deleted the repo/main-correction branch March 14, 2026 13:14
@GuilhermeF03 GuilhermeF03 added enhancement New feature or request pull-request labels Mar 15, 2026
@GuilhermeF03 GuilhermeF03 added this to the 0.1.0 - Headless Version milestone Mar 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request pull-request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants