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

Conversion of UnresolvedConstructor to real constructor isn't instrumented #9381

Closed
JaroslavTulach opened this issue Mar 12, 2024 · 7 comments · Fixed by #9452
Closed

Conversion of UnresolvedConstructor to real constructor isn't instrumented #9381

JaroslavTulach opened this issue Mar 12, 2024 · 7 comments · Fixed by #9452
Assignees
Labels
-compiler -language-server p-high Should be completed in the next sprint

Comments

@JaroslavTulach
Copy link
Member

As pointed by Dmitry at
#9293 (comment)

The invocation that turns autoscoped constructor into real constructor is not instrumented. As such it cannot be observed by the IDE.

@JaroslavTulach JaroslavTulach added p-high Should be completed in the next sprint -compiler labels Mar 12, 2024
@JaroslavTulach JaroslavTulach self-assigned this Mar 12, 2024
@JaroslavTulach JaroslavTulach moved this from ❓New to ⚙️ Design in Issues Board Mar 15, 2024
@JaroslavTulach JaroslavTulach moved this from ⚙️ Design to 🔧 Implementation in Issues Board Mar 19, 2024
@enso-bot
Copy link

enso-bot bot commented Mar 20, 2024

Jaroslav Tulach reports a new STANDUP for yesterday (2024-03-19):

Progress: - debugging autoscoped constructors

  • planning meeting
  • initial version integrated: 5d6eed9
  • playing with the IDE and autoscoped constructor It should be finished by 2024-03-27.

Next Day: On vacation

@enso-bot
Copy link

enso-bot bot commented Mar 21, 2024

Jaroslav Tulach reports a new STANDUP for yesterday (2024-03-20):

Progress: - Better location for ..A: d439217#r1531508469

Next Day: On vacation

GitHub
Pull Request Description So far reproduces and later will fix #9381. Checklist Please ensure that the following checklist has been satisfied before submitting the PR:

The documentation has been u...

GitHub
Pull Request Description So far reproduces and later will fix #9381. Checklist Please ensure that the following checklist has been satisfied before submitting the PR:

The documentation has been u...

@enso-bot
Copy link

enso-bot bot commented Mar 25, 2024

Jaroslav Tulach reports a new STANDUP for the last Friday (2024-03-22):

Progress: - investigated Decimal != Integer: #9462 (comment)

Next Day: Catch up after vacation

@enso-bot
Copy link

enso-bot bot commented Mar 26, 2024

Jaroslav Tulach reports a new STANDUP for yesterday (2024-03-25):

Progress: - downloading bug: #9527

Next Day: Polish FQN vs. imports PR

Discord
Discord is the easiest way to communicate over voice, video, and text. Chat, hang out, and stay close with your friends and communities.

@JaroslavTulach
Copy link
Member Author

Some instrumentation messages get to language server, but then they get swallowed. @4e6, can you please take a look and evaluate? Thank you. Feel free to reassign back when more work in engine is needed.

@enso-bot
Copy link

enso-bot bot commented Mar 27, 2024

Jaroslav Tulach reports a new STANDUP for yesterday (2024-03-26):

Progress: - FQNs Delimited problem and solution: #9539 (comment)

Next Day: Integrate FQN vs. imports PR

Discord
Discord is the easiest way to communicate over voice, video, and text. Chat, hang out, and stay close with your friends and communities.

@enso-bot
Copy link

enso-bot bot commented Mar 28, 2024

Jaroslav Tulach reports a new STANDUP for yesterday (2024-03-27):

Progress: - analyzing failures in FQN PR: #9539

  • creating ImportResolverAlgorithm: 1c9bdc4
  • refactored ImportResolverAlgorithm: c54b8cd
  • standup, meetings It should be finished by 2024-03-27.

Next Day: Integrate FQN vs. imports PR

@JaroslavTulach JaroslavTulach assigned JaroslavTulach and unassigned 4e6 Apr 2, 2024
@JaroslavTulach JaroslavTulach moved this from 🔧 Implementation to 👁️ Code review in Issues Board Apr 3, 2024
@mergify mergify bot closed this as completed in #9452 Apr 3, 2024
mergify bot pushed a commit that referenced this issue Apr 3, 2024
@github-project-automation github-project-automation bot moved this from 👁️ Code review to 🟢 Accepted in Issues Board Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-compiler -language-server p-high Should be completed in the next sprint
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants