Skip to content

docs: Adjust globals documentation#1100

Merged
dipinknair merged 10 commits into
mainfrom
docs/update-globals-2
Mar 3, 2025
Merged

docs: Adjust globals documentation#1100
dipinknair merged 10 commits into
mainfrom
docs/update-globals-2

Conversation

@klmcadams

Copy link
Copy Markdown
Contributor
  • Clarify when to use app.update_globals(globals()) vs. app = App(globals=globals()) (If you want to exclude enums in global vars, you should use app.update_globals(globals(), False)
  • Update cheatsheet with globals parameter

@klmcadams klmcadams requested a review from a team as a code owner February 20, 2025 21:24
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Feb 20, 2025
@codecov

codecov Bot commented Feb 20, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.90%. Comparing base (46a4f1a) to head (238c48c).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1100      +/-   ##
==========================================
+ Coverage   79.71%   79.90%   +0.18%     
==========================================
  Files          43       43              
  Lines        2854     2856       +2     
==========================================
+ Hits         2275     2282       +7     
+ Misses        579      574       -5     

Comment thread doc/source/conf.py Outdated
Comment thread doc/source/conf.py Outdated
@dipinknair dipinknair merged commit da14f6a into main Mar 3, 2025
@dipinknair dipinknair deleted the docs/update-globals-2 branch March 3, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants