Skip to content
This repository has been archived by the owner on Nov 23, 2024. It is now read-only.

feat: scope detection #120

Merged
merged 65 commits into from
Jun 22, 2023
Merged

feat: scope detection #120

merged 65 commits into from
Jun 22, 2023

Conversation

lukarade
Copy link
Contributor

@lukarade lukarade commented May 24, 2023 •
edited
Loading

Summary of Changes

  • implemented a feature to detect the scope of classes, functions, calls and variables
  • works with arbitrarily complicated nested modules
  • it generates a tree, where each node consists of its astroid node, its children and its parents

An example can be seen below:
grafik

Testing instructions

  • run test_get_scope

lukarade and others added 30 commits April 7, 2023 03:30
@lukarade
test: added test for get_name_nodes
4600e4a
@lukarade
feat: added get_name_nodes to extract Name/AssignName nodes for fur…
0b08adc 
…ther analysis
@lukarade
test: added and improved testcases for test_get_name_nodes
037da7b
@lukarade
test: changed the expected value to better represent the returns of t…
fb0bcbb 
…he function
@lukarade
feat: detect names within Calls
7e8b335
@lukarade
feat: (WIP) resolve_reference
9f9e428
@lukarade
feat: resolve_reference for target and value name nodes
d91b536
@lukarade
test: added tests for calc_node_id
1739a66
@lukarade
feat: added function calc_node_id
8a521e3
@lukarade
test: added tests for construct_reference_list
4ac7cc6
@lukarade
fix: cleanup
9e73e0a
@lukarade
feat: added a simple implementation for Scope of nodes
4c965a4
@lukarade
feat: added a better implementation for Scope of nodes
a6e2166
@lukarade
test: added test for get_nodes_for_scope
264019c
@lukarade
feat: added get_nodes_for_scope to return a list of all Nodes withi…
e667812 
…n a scope
@lukarade
fix: fixed a bug where names on module and class scope were not detected
54ac89f
@lukarade
fix: removed usage field from Reference
2e73e5b
@lukarade
test: changed attribute and instance tests and added testcases for im…
11fd5ff 
…port
@lukarade
feat: added detection of simple attributs and instances
5828ef6
@lukarade
feat: added detection of chained attributes as value
76305be
@lukarade
fix: test can now deal with results from more than one node
dc8e29f
@lukarade
feat: added MemberAccess detection for AssignAttr
5723523
@lukarade
test: added tests for Scope detection
3c33342
@lukarade
feat: (WIP) added detection of Module, Class and Function Scope
826f660
@lukarade
feat: (WIP) added Scope detection of Import, now calculate Scope usin…
e3f22bc 
…g a stack
@lukarade
feat: added Scope detection for MemberAccess, fixed some bugs, adde…
930b139 
…d typing
@lukarade
feat: (WIP) reworked scope analysis
1dde3b5
@lukarade
test: adapted test cases to new scoping
a50ce17
@lukarade
feat: (WIP) resolve_reference
bfaec44
@lukarade
feat: (WIP) redone get_scope
fabdd4b
@lukarade lukarade marked this pull request as ready for review May 25, 2023 10:40
@lars-reimann lars-reimann marked this pull request as draft May 25, 2023 12:31
@lars-reimann lars-reimann marked this pull request as ready for review May 30, 2023 13:32
lars-reimann
lars-reimann reviewed May 30, 2023
View reviewed changes
package-lock.json Outdated Show resolved Hide resolved
@lukarade lukarade requested a review from a team June 11, 2023 12:05
@lars-reimann lars-reimann merged commit 410131a into main Jun 22, 2023
@lars-reimann lars-reimann deleted the 90-function-to-resolve-references branch June 22, 2023 12:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@nvollroth nvollroth nvollroth left review comments

@lars-reimann lars-reimann lars-reimann approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Function to resolve references
4 participants
@lukarade @lars-reimann @nvollroth @megalinter-bot