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

GH-103082: Rename PY_MONITORING_EVENTS to _PY_MONITORING_EVENTS #107069

Merged
merged 1 commit into from
Jul 22, 2023

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Jul 22, 2023

Rename private C API constants:

  • Rename PY_MONITORING_UNGROUPED_EVENTS to _PY_MONITORING_UNGROUPED_EVENTS
  • Rename PY_MONITORING_EVENTS to _PY_MONITORING_EVENTS

Rename private C API constants:

* Rename PY_MONITORING_UNGROUPED_EVENTS to _PY_MONITORING_UNGROUPED_EVENTS
* Rename PY_MONITORING_EVENTS to _PY_MONITORING_EVENTS
@vstinner
Copy link
Member Author

_Py_Monitors, _PyCoMonitoringData, PyCodeObject._co_monitoring are private, so mark also the two associate constants as private.

@vstinner vstinner enabled auto-merge (squash) July 22, 2023 21:06
@vstinner vstinner merged commit 0927a2b into python:main Jul 22, 2023
18 checks passed
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry, @vstinner, I could not cleanly backport this to 3.12 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 0927a2b25c059988e237108605ed8ab0c5459c53 3.12

@vstinner vstinner deleted the pymonitor branch July 22, 2023 21:37
@bedevere-bot
Copy link

GH-107075 is a backport of this pull request to the 3.12 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.12 bug and security fixes label Jul 22, 2023
vstinner added a commit to vstinner/cpython that referenced this pull request Jul 22, 2023
…python#107069)

Rename private C API constants:

* Rename PY_MONITORING_UNGROUPED_EVENTS to _PY_MONITORING_UNGROUPED_EVENTS
* Rename PY_MONITORING_EVENTS to _PY_MONITORING_EVENTS

(cherry picked from commit 0927a2b)
vstinner added a commit that referenced this pull request Jul 22, 2023
#107069) (#107075)

GH-103082: Rename PY_MONITORING_EVENTS to _PY_MONITORING_EVENTS (#107069)

Rename private C API constants:

* Rename PY_MONITORING_UNGROUPED_EVENTS to _PY_MONITORING_UNGROUPED_EVENTS
* Rename PY_MONITORING_EVENTS to _PY_MONITORING_EVENTS

(cherry picked from commit 0927a2b)
jtcave pushed a commit to jtcave/cpython that referenced this pull request Jul 23, 2023
…python#107069)

Rename private C API constants:

* Rename PY_MONITORING_UNGROUPED_EVENTS to _PY_MONITORING_UNGROUPED_EVENTS
* Rename PY_MONITORING_EVENTS to _PY_MONITORING_EVENTS
mementum pushed a commit to mementum/cpython that referenced this pull request Jul 23, 2023
…python#107069)

Rename private C API constants:

* Rename PY_MONITORING_UNGROUPED_EVENTS to _PY_MONITORING_UNGROUPED_EVENTS
* Rename PY_MONITORING_EVENTS to _PY_MONITORING_EVENTS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants