Skip to content

1.20: finalize python change notes#1127

Merged
markshannon merged 3 commits into
github:rc/1.20from
felicitymay:1.20/python-change-notes
Mar 19, 2019
Merged

1.20: finalize python change notes#1127
markshannon merged 3 commits into
github:rc/1.20from
felicitymay:1.20/python-change-notes

Conversation

@felicitymay

Copy link
Copy Markdown
Contributor

This pull request is intended to finalize the change notes for Python 1.20. Much of the change is to bring information from the bottom of the page into the "General improvements" section at the top.

After discussion with Mark, I've reworded one section. I hope that I've avoided changing the meaning or introducing any errors.

@markshannon or @taus-semmle - do you have time to review these changes?
@jf205 - mostly for information.

Comment thread change-notes/1.20/analysis-python.md Outdated
Removes false positives seen when using Python 3.6, but not when using earlier versions.
The API has been improved to declutter the global namespace and improve discoverability and readability.
* New predicates `ModuleObject::named(name)` and `ModuleObject.attr(name)` have been added, allowing more readable access to common objects. For example, `(any ModuleObject m | m.getName() = "sys").getAttribute("exit")` can be replaced with `ModuleObject::named("sys").attr("exit")`
* The API for accessing builtin functions has been improved. Predicates of the form `theXXXFunction()`, such as `theLenFunction()`, have been deprecated in favour of `Object::builtin(name)`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be US English favor instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @taus-semmle - I missed that entirely...

@taus-semmle taus-semmle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@felicitymay

Copy link
Copy Markdown
Contributor Author

@markshannon - is this ready to merge or is there anything you want me to update?

@markshannon markshannon merged commit 29c4e27 into github:rc/1.20 Mar 19, 2019
@felicitymay felicitymay deleted the 1.20/python-change-notes branch September 23, 2019 16:31
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