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.

Sign up for GitHub

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

Jump to bottom

Reorganize visitor/mapper code #283

Closed
JonoYang opened this issue Feb 5, 2024 · 2 comments
Closed

Reorganize visitor/mapper code #283

JonoYang opened this issue Feb 5, 2024 · 2 comments

Comments

@JonoYang
Copy link
Contributor

JonoYang commented Feb 5, 2024

Currently we are putting code for the on-demand/priority queue in with the visitor code. It would be better if we separated the code. We should also decide which term we want to use for the on-demand/priority queue, since we use both currently. It is also worth considering if we want to merge the visitor and mapper code as well.
JonoYang added a commit that referenced this issue Feb 6, 2024
@JonoYang
Move maven priority queue related code to minecode.collectors #283
e0cb689 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 6, 2024
@JonoYang
Move npm code #283
49f0f6c 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 17, 2024
@JonoYang
Move maven priority queue related code to minecode.collectors #283
22577ee 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 17, 2024
@JonoYang
Move npm code #283
6eb65ca 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 20, 2024
@JonoYang
Move maven priority queue related code to minecode.collectors #283
779c4fd 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 20, 2024
@JonoYang
Move npm code #283
4b270e2 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 22, 2024
@JonoYang
Move maven priority queue related code to minecode.collectors #283
c26399c 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 22, 2024
@JonoYang
Move npm code #283
000d2a8 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 27, 2024
@JonoYang
Move maven priority queue related code to minecode.collectors #283
13d0212 
Signed-off-by: Jono Yang <[email protected]>
JonoYang added a commit that referenced this issue Feb 27, 2024
@JonoYang
Move npm code #283
ada55ea 
Signed-off-by: Jono Yang <[email protected]>
@pombredanne
Copy link
Contributor

See also #521

@pombredanne pombredanne mentioned this issue Aug 7, 2024
Open
@JonoYang
Copy link
Contributor Author

The visitors and mappers have been merged into a single module named miners.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pombredanne @JonoYang